Skip to content

III-5499 - Show alert when you just edited an event#816

Merged
brampauwelyn merged 4 commits intomainfrom
feature/III-5499
Nov 5, 2025
Merged

III-5499 - Show alert when you just edited an event#816
brampauwelyn merged 4 commits intomainfrom
feature/III-5499

Conversation

@brampauwelyn
Copy link
Contributor

@brampauwelyn brampauwelyn commented Nov 4, 2025

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds an informational alert message that displays after an event has been edited. The alert informs users that their changes have been saved and will be visible on UiTinVlaanderen within an hour.

  • Added query parameter detection to show an edit confirmation alert
  • Implemented multilingual support for the alert message in Dutch, French, and German
  • Positioned the alert below the tab navigation on the event detail page

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/event-detail/ui/event-detail.html Added alert div to display edit confirmation message
src/event-detail/ui/event-detail.controller.js Added logic to detect edited=true query parameter
src/core/translations/dutch-translations.constant.js Added Dutch translation for edit alert message
src/core/translations/french-translations.constant.js Added French translation for edit alert message
src/core/translations/german-translations.constant.js Added German translation for edit alert message
dist/udb3-angular.js Distribution file with compiled changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@brampauwelyn brampauwelyn merged commit ea9eb5b into main Nov 5, 2025
7 checks passed
@brampauwelyn brampauwelyn deleted the feature/III-5499 branch November 5, 2025 13:17
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