Skip to content

feat/#312: Make Notification component configurable

dindigala requested to merge feature/#312-Notification into develop

Feature

Summary

This MR provides:

  • Refactoring of Notification component to display different messages, icons and their colors.

For now i have added only two messages - info , warning but we can more.

Does the result of the MR comply to our "definition of done"?

  • Unit tests passed
  • Code reviewed
  • Acceptance criteria met
  • Functional tests passed
  • Non-Functional requirements met
  • Product Owner accepts the User Story

Documentation

  • I updated the README (if applicable)
  • I provided my functions with appropriate documentation
  • I adjusted other parts of the documentation (if applicable)

Tests

Are we able to test this new feature?

  • Yes, everything can be done via unit tests.
  • Yes, you can test by following these steps: Please pull the changes into local to test different message.

Try changing the variant-type in annotation component from info to warning and can observe that icon is change to exclamantion triangle warning

EX: Testdata Manifest 003 - Sheet 2

If needed we can add different messages and icons notification component.

  • No, it is not possible.

Changelog

  • I added a statement to the CHANGELOG.

Related Tickets

Add all related issues and especially those to be closed.

Related

Closes

Logs and Screenshots

/cc Mathias Göbel, Frank Schneider, Michelle Weidling

Edited by dindigala

Merge request reports