-
Notifications
You must be signed in to change notification settings - Fork 151
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 documentation for Emails and Messages app #821
Conversation
krzysztofwolski
commented
Jul 6, 2023
- Add Emails and Messages documentation
- Sort app cards on the overview page
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
- use only the specific channels | ||
- use all channels except the specific ones | ||
|
||
## Development |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably worth linking https://docs.saleor.io/docs/3.x/developer/app-store/development
and leave here only app specific stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: blocking
polish: non-blocking
suggestion: should have blocking/non-blocking labels
|
||
## Supported events | ||
|
||
The application is listening to the following events: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question:
What type of events are these? Because I don't see them on the webhook events list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The list was copied from old plugin.
Reorganized this section to contain more info on where those events come from and added links for non-trivial topics like activating the account. WDYT?
docs/developer/app-store/apps/emails-and-messages/troubleshooting.mdx
Outdated
Show resolved
Hide resolved
@krzysztofwolski could we move this forward? |
Co-authored-by: Adrian Pilarczyk <adrianpilarczyk314@gmail.com>
Co-authored-by: Adrian Pilarczyk <adrianpilarczyk314@gmail.com>
docs/developer/app-store/apps/emails-and-messages/troubleshooting.mdx
Outdated
Show resolved
Hide resolved
docs/developer/app-store/apps/emails-and-messages/troubleshooting.mdx
Outdated
Show resolved
Hide resolved
docs/developer/app-store/apps/emails-and-messages/troubleshooting.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Adrian Pilarczyk <adrianpilarczyk314@gmail.com>
@zaiste code owner review required to merge. Could you take a look? |
|
||
1. Create an account at [MailSlurp](https://www.mailslurp.com/) | ||
2. Create a new inbox - [documentation](https://www.mailslurp.com/guides/creating-inboxes/) | ||
3. Use the inbox address to create a new sender in Sendgrid - [documentation](https://docs.sendgrid.com/ui/sending-email/senders) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
polish: I think it looks nicer to link to the docs right from the corresponding fragment.
3. Use the inbox address to create a new sender in Sendgrid - [documentation](https://docs.sendgrid.com/ui/sending-email/senders) | |
3. Use the inbox address to [create a new sender in Sendgrid](https://docs.sendgrid.com/ui/sending-email/senders) |