[MGNLUI-6613] NotificationPropertySetFactory fails on unknown field Created: 11/Mar/21 Updated: 25/Oct/21 Resolved: 20/Oct/21 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | 6.2.11 |
| Fix Version/s: | 6.2.13 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Carlos Cantalapiedra | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | maintenance, ui-team-support | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: | |||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[X]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[X]*
Steps to reproduce, expected, and actual results filled
[X]*
Affected version filled
|
||||||||
| Release notes required: |
Yes
|
||||||||
| Date of First Response: | |||||||||
| Sprint: | UI FW 39 | ||||||||
| Story Points: | 3 | ||||||||
| Description |
DescriptionAfter falling with an exception in Configuration App, if you click at the "Read more" button within the Warning Notification, it throws a Sub-app failed to start: detail error Steps to reproduce
Expected resultsAfter click on "Read more" the full Notification can be read at the Notification detail section of the Notifications app Actual resultsThe app fails to start (check notifications-app-fail.png java.lang.IllegalArgumentException: Could not resolve property name comment from info.magnolia.admincentral.apps.notifications.NotificationPropertySetFactory$1@1f072584 at WorkaroundN/A Development notesWhen accessing the notification detail from other app, like asset, it works. E.g, try to publish an asset that fails (public instance not accesible), then the notification appears and if you click the "Read more" button, it goes to the Notifications app and shows the full notification |
| Comments |
| Comment by Roman Kovařík [ 22/Oct/21 ] |
|
For release notes: notification app detail doesn't crash anymore on unknown field name. |