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

Show notifications in GUI #3015

Open
mzueva opened this issue Jan 17, 2023 · 2 comments
Open

Show notifications in GUI #3015

mzueva opened this issue Jan 17, 2023 · 2 comments
Assignees
Labels
kind/enhancement New feature or request state/has-doc Issues that have documentation

Comments

@mzueva
Copy link
Contributor

mzueva commented Jan 17, 2023

Background
At the moment Cloud Pipeline platform sends notifications to users via email. For many cases it would be useful to show such notifications in the GUI as well.

Approach

  • A new API methods shall be added to manage UserNotification entities (userId, subject, text, dateCreated, (un)read flag, dateRead). CRUD operation shall be implemented to create, delete, get list of notifications for a user.
  • All email notifications shall be duplicated into this new object. I'd suggest to do this somewhere at NotificationManager level.
  • Notifications shall be deleted by a scheduled service after a period, configured via SystemPreferences.
@mzueva mzueva added the kind/enhancement New feature or request label Jan 17, 2023
@mzueva mzueva changed the title GUI Notifications Show notifications in GUI Jan 17, 2023
mzueva added a commit that referenced this issue Jan 27, 2023
mzueva added a commit that referenced this issue Jan 27, 2023
mzueva added a commit that referenced this issue Feb 6, 2023
* Issue 3015: Show notifications in GUI - pagination + read all

* Issue 3015: Show notifications in GUI - pagination + read all

* Issue 3015: Show notifications in GUI - on/off ui notifications

* Issue #3015 User notifications: add env var to enable UI notifications

---------

Co-authored-by: mzueva <mariia_zueva@epam.com>
@maryvictol
Copy link
Collaborator

@AleksandrGorodetskii

Bug
Notifications table: when the user changes the notifications type from New messages to Read messages (or vice versa) the application tries to open the same table page number for Read messages table that was opened in the New messages table. If there are fewer pages in the Read messages table, then an empty table is shown.

mzueva added a commit that referenced this issue Mar 14, 2023
mzueva added a commit that referenced this issue Mar 14, 2023
mzueva added a commit that referenced this issue Mar 14, 2023
* Issue 3015: Show notifications in GUI - pagination + read all

* Issue 3015: Show notifications in GUI - pagination + read all

* Issue 3015: Show notifications in GUI - on/off ui notifications

* Issue #3015 User notifications: add env var to enable UI notifications

---------

Co-authored-by: mzueva <mariia_zueva@epam.com>
mzueva added a commit that referenced this issue Mar 14, 2023
NShaforostov added a commit that referenced this issue Mar 17, 2023
* Doc 'Creation of SampleSet metadata item via GUI' was added
* Doc 'Environments synchronization' was added
* 'Enironments sync' doc was added to RN0.17
* (Issues #3020, #3028) Docs added
* (Issue #3080) Billing docs and RN update
* (Issue #3015) Push notifications docs were added
@NShaforostov
Copy link
Collaborator

Docs were added via #2919 and located here.

@NShaforostov NShaforostov added the state/has-doc Issues that have documentation label Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request state/has-doc Issues that have documentation
Projects
None yet
Development

No branches or pull requests

5 participants