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

unskip sms verification e2e test #3114

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

joeyorlando
Copy link
Contributor

@joeyorlando joeyorlando commented Oct 4, 2023

What this PR does

Since we now have a Mailslurp account we can start running this test. At the moment it is only run 1x daily on a GitHub Actions cron (to avoid incurring costs; it is skipped via a Playwright "test annotation filter").

MAILSLURP_API_KEY repo secret has already been updated.

Closes #2693

@joeyorlando joeyorlando added pr:no changelog pr:no public docs Added to a PR that does not require public documentation updates labels Oct 4, 2023
@joeyorlando joeyorlando requested a review from a team October 4, 2023 12:54
@joeyorlando joeyorlando merged commit d6c0150 into dev Oct 4, 2023
@joeyorlando joeyorlando deleted the jorlando/add-mailslurp-credentials branch October 4, 2023 13:04
brojd pushed a commit that referenced this pull request Sep 18, 2024
# What this PR does

Since we now have a Mailslurp account we can start running this test. At
the moment it is only run 1x daily on a GitHub Actions cron (to avoid
incurring costs; [it is
skipped](https://github.com/grafana/oncall/blob/dev/.github/workflows/linting-and-tests.yml#L317)
via a Playwright "[test annotation
filter](https://github.com/grafana/oncall/blob/dev/grafana-plugin/package.json#L14-L15)").

`MAILSLURP_API_KEY` repo secret has already been updated.

Closes #2693
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uncomment out/finish e2e test for SMS verification/alert notification workflow
1 participant