Use toast for preferences saved notification

changed milestone to %16.8

added Pajamas UX component:alert component:toast devopsgovern groupauthentication sectionsec settings user settings labels

assigned to @clavimoniere

added UX Paper Cuts label

changed the description

added frontend typebug labels

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
frontend	`@aalakkad` (UTC+3, 8 hours ahead of author)	`@justin_ho` (UTC+7, 12 hours ahead of author)
test for `spec/features/*`	`@hmuralidhar` (UTC+11, 16 hours ahead of author)	Maintainer review is optional for test for `spec/features/*`

Please check reviewer's status!

Reviewer is available!
Reviewer is unavailable!

Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits fada90d7 and 9821f690

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	4.12 MB	4.12 MB	-	0.0 %
mainChunk	3.09 MB	3.09 MB	-	0.0 %

Note: We do not have exact data for fada90d7. So we have used data from: e5c4b45e.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.

Please look at the full report for more details

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for 9821f690

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Data Stores | 23     | 0      | 0       | 0     | 23    | ✅     |
| Create      | 54     | 0      | 7       | 0     | 61    | ✅     |
| Verify      | 31     | 0      | 0       | 0     | 31    | ✅     |
| Govern      | 72     | 0      | 0       | 0     | 72    | ✅     |
| Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
| Plan        | 55     | 0      | 0       | 0     | 55    | ✅     |
| Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
| Package     | 15     | 0      | 1       | 0     | 16    | ✅     |
| Monitor     | 7      | 0      | 0       | 0     | 7     | ✅     |
| Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 264    | 0      | 9       | 0     | 273   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: test report for 9821f690

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Govern      | 292    | 0      | 20      | 10    | 312   | ✅     |
| Create      | 149    | 0      | 16      | 2     | 165   | ✅     |
| Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
| Plan        | 8      | 0      | 0       | 0     | 8     | ✅     |
| Package     | 0      | 0      | 2       | 0     | 2     | ➖     |
| Data Stores | 4      | 0      | 0       | 0     | 4     | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 461    | 0      | 38      | 12    | 499   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+

e2e-review-qa: test report for 748fad9d

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Govern           | 61     | 0      | 2       | 0     | 63    | ✅     |
| Create           | 6      | 0      | 4       | 0     | 10    | ✅     |
| Data Stores      | 2      | 0      | 0       | 0     | 2     | ✅     |
| Plan             | 3      | 0      | 1       | 0     | 4     | ✅     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 76     | 0      | 9       | 0     | 85    | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+

added 1 commit

748fad9d - Use toast for preferences saved notification

Compare with previous version

@seggenberger can you do a groupux paper cuts review on this? This is my first time doing anything with toasts.

requested review from @seggenberger

approved this merge request

@seggenberger, thanks for approving this merge request.

This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break master, a new pipeline will be started shortly.

Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.