Skip to content

feat: add enable_email field to app.yaml configuration #5539

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: add enable_email field to app.yaml configuration #5539

wants to merge 2 commits into from

Conversation

ayys
Copy link
Contributor

@ayys ayys commented Apr 28, 2025

this allows users to specify if they want an email address associated
with their app. When enabled, the following happens:

  • a randomly generated email address is associated with the app
  • env vars are set to allow the app to access the email credentials

@ayys
feat: add enable_email field to app.yaml configuration
0c91ab0 
this allows users to specify if they want an email address associated
with their app. When enabled, the following happens:

- a randomly generated email address is associated with the app
- env vars are set to allow the app to access the email credentials
@ayys ayys requested a review from syrusakbary as a code owner April 28, 2025 13:03
@ayys ayys requested a review from theduke April 28, 2025 13:04
@promptless promptless bot mentioned this pull request Apr 28, 2025
Open
@promptless Promptless
Copy link

promptless bot commented Apr 28, 2025

📝 Documentation updates detected! A separate PR for documentation updates has been made here: wasmerio/docs.wasmer.io#132

theduke
theduke reviewed May 7, 2025
View reviewed changes
Copy link
Contributor

@theduke theduke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woudln't this be better as a capability, since we may need a bunch of custom additional settings in the future?

capabilities:
  email:
    enabled: true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theduke theduke theduke left review comments

@syrusakbary syrusakbary Awaiting requested review from syrusakbary syrusakbary is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ayys @theduke