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

Add new endpoints to set a process / state as notified/not notified for all user #6660

Closed
freddidierRTE opened this issue Jun 17, 2024 · 0 comments · Fixed by #6703
Closed
Assignees
Labels
Enhancement New feature

Comments

@freddidierRTE
Copy link
Contributor

freddidierRTE commented Jun 17, 2024

We need to be able to create a new process/state having no user receiving the process/state by default

To do that the idea is to provide an endpoint to set ProcessStatesNotNotified for all users with one request

The end point will be from the service point

  • to remove notification for all users : DELETE /notificationConfiguration/processStateNotified
  • to add notification for all users : POST /notificiationConfiguration/processStateNotified

The body shall contains the processId and the state to modify

The modification will address only the existing user settings , it a user has no settings, no settings is to be created

Should be tested via karate tests

Add a command line in cli to access this endpoint , the endpoint shall be access via nginx (port 2002)

opfab users notif-config  set-notified <process> <state>
opfab users notif-config  set-not-notified <process> <state>

The same is to be done for ProcessesStatesNotifiedByEmail

End point : /notificationConfiguration/processStateNotifiedByMail
cli :
opfab users notif-config set-not-notified-mail ...
opfab users notif-config set-notified-mail

@freddidierRTE freddidierRTE added the Enhancement New feature label Jun 17, 2024
@freddidierRTE freddidierRTE changed the title Add a new endoints to set a notification as notified/not notified for all user Add new endoints to set a notification as notified/not notified for all user Jun 17, 2024
@freddidierRTE freddidierRTE changed the title Add new endoints to set a notification as notified/not notified for all user Add new endpoints to set a process / state as notified/not notified for all user Jun 19, 2024
@quinarygio quinarygio self-assigned this Jun 24, 2024
quinarygio added a commit that referenced this issue Jun 25, 2024
quinarygio added a commit that referenced this issue Jun 25, 2024
quinarygio added a commit that referenced this issue Jun 26, 2024
quinarygio added a commit that referenced this issue Jun 27, 2024
quinarygio added a commit that referenced this issue Jun 28, 2024
freddidierRTE pushed a commit that referenced this issue Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature
Projects
None yet
2 participants