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

Release 2024.35.1 #1398

Merged
merged 10 commits into from
Aug 28, 2024
Merged

Release 2024.35.1 #1398

merged 10 commits into from
Aug 28, 2024

Conversation

rasben
Copy link
Contributor

@rasben rasben commented Aug 28, 2024

No description provided.

Dresse and others added 10 commits August 1, 2024 10:59
Also added an extra space between the asterisk to comply with how
required fields are indidcated in the rest of the project.
Since the cms is handling those cases we do not need to take them into
consideration
Otherwise we will get a missing text error when developing
The initial implementation was unclear and buggy. So we reduced the
amount of states and cleaned up the code.
Make sure that it is remembered across page loads
This has been used in a previous version of the apps but are not in use
anymore.
Make sure that the persisted redux states are cleared before logging
out.
Since the behaviour has changed we need to update the tests accordingly.
…-remove-null-from-pickup-branch

Updated the required check to display empty string instead of null.
…-only-show-blocked-modal-when-reserving

DDFLSBP-621 - Make sure to only show blocked modal when logging in
@rasben rasben merged commit 801bd18 into main Aug 28, 2024
26 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.

4 participants