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

refactor: Update api usage notification sending #4214

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

zachaysan
Copy link
Contributor

Thanks for submitting a PR! Please check the boxes below:

  • I have run pre-commit to check linting
  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Tweak the language around sending API usage

How did you test this code?

Ran existing tests that send mail to confirm the call still works.

@zachaysan zachaysan requested a review from a team as a code owner June 21, 2024 15:49
@zachaysan zachaysan requested review from khvn26 and removed request for a team June 21, 2024 15:49
Copy link

vercel bot commented Jun 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Jun 21, 2024 3:49pm
flagsmith-frontend-preview ⬜️ Ignored (Inspect) Jun 21, 2024 3:49pm
flagsmith-frontend-staging ⬜️ Ignored (Inspect) Jun 21, 2024 3:49pm

@github-actions github-actions bot added api Issue related to the REST API fix labels Jun 21, 2024
@zachaysan zachaysan requested a review from matthewelwell June 21, 2024 15:50
@matthewelwell matthewelwell changed the title fix: Update api usage notification sending refactor: Update api usage notification sending Jun 21, 2024
Copy link
Contributor

github-actions bot commented Jun 21, 2024

Uffizzi Preview deployment-53347 was deleted.

@zachaysan zachaysan added this pull request to the merge queue Jun 21, 2024
Merged via the queue into main with commit 5bafb4c Jun 21, 2024
23 checks passed
@zachaysan zachaysan deleted the fix/update_api_usage_notification_sending branch June 21, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Issue related to the REST API fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants