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

How to add FromAddress, Identity and RoleArn To Environment - Can't Remove Email Channel #11768

Closed
mjmostachetti opened this issue Jan 13, 2023 · 9 comments
Assignees
Labels
bug Something isn't working notifications Issues tied to the notifications category p1

Comments

@mjmostachetti
Copy link

mjmostachetti commented Jan 13, 2023

Amplify CLI Version

10.6.1

Question

I ran this ----

amplify update notifications
✔ Choose the notification channel to configure · Email
The Email channel is currently enabled
✔ Do you want to disable the Email channel (y/N) · yes
🛑 FromAddress, Identity or RoleArn is missing for the Email channel

Resolution: Provide the required parameters for the Email channel
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/

Session Identifier: 3b2c9940-05e1-4947-875d-a56a7a421879


I didn't add the email channel myself, but when I run 'amplify update notifications' I don't see a prompt for any of these things.

amplify update notifications
✔ Choose the notification channel to configure · Email
The Email channel is currently enabled
✔ Do you want to disable the Email channel (y/N) · no
✔ The 'From' Email address used to send emails · MYEMAIL
✔ The ARN of an identity verified with SES · MYARN
✔ The ARN of an IAM Role used to submit events to Mobile notifications' event ingestion service ·
✔ The Email channel has been successfully updated.

Where do I add them?

@mjmostachetti mjmostachetti added pending-triage Issue is pending triage question General question labels Jan 13, 2023
@mjmostachetti
Copy link
Author

mjmostachetti commented Jan 13, 2023

This is interesting.
#5100

@josefaidt
Copy link
Contributor

Hey @mjmostachetti 👋 thanks for raising this! While we have a few active bugs for notifications, it appears the missing inputs are coming from the following prompts:

✔ The 'From' Email address used to send emails · MYEMAIL
✔ The ARN of an identity verified with SES · MYARN
✔ The ARN of an IAM Role used to submit events to Mobile notifications' event ingestion service ·

Do you see this information captured in amplify/team-provider-info.json or amplify/backend/backend-config.json?

@josefaidt josefaidt added notifications Issues tied to the notifications category pending-response Issue is pending response from the issue author labels Jan 13, 2023
@mjmostachetti
Copy link
Author

None of my inputs from those 3 questions are anywhere in the solution.

@github-actions github-actions bot removed the pending-response Issue is pending response from the issue author label Jan 13, 2023
@mjmostachetti
Copy link
Author

@josefaidt Are there any work arounds for this? Any ETA on a fixes for the notifications bugs?

@ykethan
Copy link
Member

ykethan commented Jan 19, 2023

Hey, I was able to reproduce the issue. marking this as bug. Looking into workarounds.

@ykethan ykethan added bug Something isn't working and removed pending-triage Issue is pending triage labels Jan 19, 2023
@josefaidt josefaidt removed the question General question label Jan 19, 2023
@fujimonfn
Copy link

Notifications (and analytics) resource by Amplify is kind of buggy...
I managed to create notifications resource on dev env, but when I try to add new environment, it doesn't work.
image

Once I run into it, I can't remove thoese resources ("amplify remove notifications" aloso fails with error "Cannot read property 'Id' of undefined")

Please help.

@ykethan
Copy link
Member

ykethan commented Jan 20, 2023

Hey @fujimonfn, To mitigate the issue I was able to find a workaround. Please find the steps as follows.

For the error: FromAddress, Identity or RoleArn is missing for the Email channel

  1. find the .vscode folder and set flag with amplify/backend/amplify-meta.json as false. As the file will be hidden in vscode.
  2. open the amplify-meta.json file under the amplify/backend directory.
  3. find notifications and under outputs add RoleArn with its value.

Then run amplify update notifications or amplify remove notifications to disable or remove notifications.
The team has been made aware into the situation on the error Missing required key ApplicationId in params. A PR is currently in place to mitigate this. Please refer to #11494

@josefaidt josefaidt added the p1 label Jan 24, 2023
@josefaidt
Copy link
Contributor

This is related/similar to #11819

@pavellazar
Copy link
Contributor

Thiis issue will be fixed with the released of this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working notifications Issues tied to the notifications category p1
Projects
None yet
Development

No branches or pull requests

5 participants