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

Add feature flag and link for system returns notifications #2644

Merged

Conversation

jonathangoulding
Copy link
Collaborator

https://eaflood.atlassian.net/browse/WATER-4716

The previous change to introduce this toggle - #2643

The previous change was based on the assumption of invitations and reminders being seperated. After discussion this is no longer the case. That journey could be built as one.

However, for now the feature flag has been updated to be more concise and reusable in relation to system notifications. And not specific to invitations.

As part of the ongoing work to migrate the legacy ui pages into the system repo. We need to redirect the user to the newly created pages / journey when it is ready.

This change adds a feature flag to the manage page for the return notifications invitations link.

When the feature flag is on it will redirect the user to the new page / journey built in system

https://eaflood.atlassian.net/browse/WATER-4716

The previous change to introduce this toggle - #2643

The previous change was based on the assumption of invitations and reminders being seperated. After discussion this is no longer the case. That journey could be built as one.

However, for now the feature flag has been updated to be more concise and reusable in relation to system notifications. And not specific to invitations.

As part of the ongoing work to migrate the legacy ui pages into the system repo. We need to redirect the user to the newly created pages / journey when it is ready.

This change adds a feature flag to the manage page for the return notifications invitations link.

When the feature flag is on it will redirect the user to the new page / journey built in system
@jonathangoulding jonathangoulding added the enhancement New feature or request label Nov 27, 2024
@jonathangoulding jonathangoulding self-assigned this Nov 27, 2024
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonathangoulding jonathangoulding merged commit a75057b into main Nov 28, 2024
4 checks passed
@jonathangoulding jonathangoulding deleted the feature-add-feature-toggle-for-system-notifications branch November 28, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants