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

Dismissible notification feature stops working in 5.x latest and SP 6.5.18+ #3355

Closed
3 tasks done
janpop opened this issue Jun 5, 2024 · 1 comment
Closed
3 tasks done
Assignees
Labels
Milestone

Comments

@janpop
Copy link

janpop commented Jun 5, 2024

Required Information

  • AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM 6.5.20 (on-premise)
  • ACS AEM Commons Version: 5.x latest
  • Reproducible on Latest? yes

Expected Behavior

Creating a dismissible notification: Notification can be dismissed and will not show up for user after he clicks on the dismiss button.

Actual Behavior

With latest 5.x ACS Commons and AEM SP 6.5.20 notifications can not be dismissed and re-appear with every page load

Steps to Reproduce

How to reproduce:
1.) Create a AEM 6.5 installation from scratch
2.) Install ACS Commons 5.x latest
3.) Create a notification a see that it can be dismissed
4.) Install SP 6.5.15 => [/] Notifications can still be dismissed
5.) Install SP 6.5.18 => [x] Notifications cannot be dismissed anymore, they re-appear with every page load
BTW: SP 6.5.20 does not solve the issue.

Links

Links to related assets, e.g. content packages containing test components

@janpop
Copy link
Author

janpop commented Jun 25, 2024

The issue is really annoying. Can someone please have a look.

davidjgonzalez added a commit to davidjgonzalez/acs-aem-commons that referenced this issue Jun 25, 2024
…issue with not being dismissible, and updated to use Coral3.
davidjgonzalez added a commit to davidjgonzalez/acs-aem-commons that referenced this issue Jun 25, 2024
davidjgonzalez added a commit that referenced this issue Jun 25, 2024
* #3355 - Fixed system notification dismissal issue
@davidjgonzalez davidjgonzalez self-assigned this Jun 25, 2024
@davidjgonzalez davidjgonzalez added this to the 6.6.2 milestone Jun 25, 2024
YegorKozlov pushed a commit to YegorKozlov/acs-aem-commons that referenced this issue Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants