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

App Approve | App Manager is not receiving the notification #370

Closed
tfjanjua opened this issue Jul 16, 2024 · 0 comments
Closed

App Approve | App Manager is not receiving the notification #370

tfjanjua opened this issue Jul 16, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@tfjanjua
Copy link
Contributor

tfjanjua commented Jul 16, 2024

Current Behavior

App Manager is not receiving the notification on an App Approval by an authorised approver but Service Manager is receiving it instead.

Expected Behavior

App Manager should received Approval of App notification instead of Service Manager.

Steps To Reproduce

  1. Log in to portal as CX-Operator.
  2. Navigate to the "App Release Process" page and Release an App (currently App Manager is not able to do so, because there is an issue, please follow the issue for more details)
  3. Navigate to the "App Management: Admin Board" page.
  4. Locate an application in "In Review" status.
  5. Click on the approval icon to approve the application.
  6. Verify if a notification is received on the portal interface (Notification) or via email.

Findings

This issue is related to the notification of an APP. App Manager is not receiving the notification if Admin approves the APP but when I investigate it so, I found that Service Manager is receiving the APP's notification instead

On further investigation, I found that wrong role Service Manager has been mentioned instead of App Manager under approveAppUserRoles param in CD repo:

appmarketplace.approveAppUserRoles.role0 : `"Sales Manager"`
appmarketplace.approveAppUserRoles.role1 : `"Service Manager"`

https://github.com/Cofinity-X/portal-charts/blob/262d6c60e56d88f5ecaea1291be3a0218e2ad215/portal/charts/values.yaml#L504

@tfjanjua tfjanjua added the bug Something isn't working label Jul 16, 2024
@tfjanjua tfjanjua added this to the Release 2.1.0 (24.08) milestone Jul 16, 2024
@tfjanjua tfjanjua added this to Portal Jul 16, 2024
@github-project-automation github-project-automation bot moved this to NEW USER REQUEST in Portal Jul 16, 2024
@tfjanjua tfjanjua moved this from NEW USER REQUEST to IN PROGRESS in Portal Jul 16, 2024
typecastcloud added a commit to typecastcloud/portal that referenced this issue Jul 16, 2024
Currently Service Manager receives notifications regarding app approval process.

Refs:  eclipse-tractusx#370
evegufy pushed a commit that referenced this issue Jul 17, 2024
Currently Service Manager receives notifications regarding app approval process.

Refs:  #370
@evegufy evegufy closed this as completed Jul 23, 2024
@github-project-automation github-project-automation bot moved this from IN PROGRESS to USER READY in Portal Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

3 participants