Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update date picker #4564

Merged
merged 7 commits into from
Jul 9, 2024
Merged

Update date picker #4564

merged 7 commits into from
Jul 9, 2024

Conversation

thecalcc
Copy link
Contributor

@thecalcc thecalcc commented Jul 5, 2024

SDBELGA-819

@thecalcc
Copy link
Contributor Author

thecalcc commented Jul 5, 2024

spellchecker failing

@tomaskikutis
Copy link
Member

spellchecker failing

You can ignore it for this PR. I pinged Petr so he reviews my fixes for those unit tests.

@thecalcc
Copy link
Contributor Author

thecalcc commented Jul 8, 2024

spellchecker failing

You can ignore it for this PR. I pinged Petr so he reviews my fixes for those unit tests.

Ok so all good to be merged? I see e2e also failing

@thecalcc thecalcc requested a review from tomaskikutis July 8, 2024 09:04
@tomaskikutis
Copy link
Member

e2e does need to pass. It might not be your PR breaking this, but take a look anyway

@tomaskikutis
Copy link
Member

I just merged the PR that fixes CI to develop, so if you update from develop your PR should pass too

@thecalcc thecalcc merged commit c5738fa into superdesk:develop Jul 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants