-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: Fetch announcement with the edit button is clicked #724
Conversation
The code changes in this commit add error handling for the case when the retrieval of an announcement fails. When the `editAnnouncement` function is called, it now makes an API request to retrieve the announcement using the `getAnnouncement` method from the `ApiService`. If the retrieval is successful, the announcement is set in the `announcementSelectionStore` and the user is redirected to the edit announcement page. However, if the retrieval fails, an error message is displayed using the `NotificationService`. This commit also includes recent user commits and recent repository commits for reference.
7de79f8
to
940047f
Compare
Quality Gate passed for 'Pay Transparency admin-frontend'Issues Measures |
Quality Gate passed for 'Pay Transparency backend-external'Issues Measures |
Quality Gate passed for 'Pay Transparency Frontend'Issues Measures |
Quality Gate passed for 'Pay Transparency doc-gen-service'Issues Measures |
Quality Gate passed for 'Pay Transparency Backend'Issues Measures |
Description
The code changes in this commit add error handling for the case when the retrieval of an announcement fails. When the
editAnnouncement
function is called, it now makes an API request to retrieve the announcement using thegetAnnouncement
method from theApiService
. If the retrieval is successful, the announcement is set in theannouncementSelectionStore
and the user is redirected to the edit announcement page. However, if the retrieval fails, an error message is displayed using theNotificationService
.Fixes # (issue)
Type of change
Checklist
Further comments
Thanks for the PR!
Deployments, as required, will be available below:
Please create PRs in draft mode. Mark as ready to enable:
After merge, new images are deployed in: