Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Webhook changed by #433

Merged
merged 1 commit into from
May 9, 2024
Merged

Webhook changed by #433

merged 1 commit into from
May 9, 2024

Conversation

laliconfigcat
Copy link
Member

Description

Webhooks now can specify ##ChangedBy## in the content and it will be replaced by the user's email and name who made the changes.

Trello card

https://trello.com/c/1rTjg9QS/228-who-changed-a-flag-webhook-variable

Requirement checklist

  • I have validated my changes on a test/local environment.
  • I have added my changes to the V1 and V2 documentations.
  • I have checked the SNYK/Dependabot reports and applied the suggested changes.
  • (Optional) I have updated outdated packages.

@laliconfigcat laliconfigcat requested review from sigewuzhere and a team as code owners May 9, 2024 14:31
@laliconfigcat laliconfigcat added ux-review The PR needs a review from the UX team text-review The PR needs a review from the text reviewers team labels May 9, 2024
Copy link

github-actions bot commented May 9, 2024

Head's up @configcat/text-reviewers - someone needs a text review!

Copy link

github-actions bot commented May 9, 2024

Head's up @configcat/ux-qa-team - someone needs a UX review!

Copy link

Deploying configcat-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b5b3870
Status: ✅  Deploy successful!
Preview URL: https://348352f9.docs-xmy.pages.dev
Branch Preview URL: https://webhookchangedby.docs-xmy.pages.dev

View logs

@laliconfigcat laliconfigcat merged commit d2ac4fa into master May 9, 2024
7 checks passed
@laliconfigcat laliconfigcat deleted the webhookchangedby branch May 9, 2024 14:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
text-review The PR needs a review from the text reviewers team ux-review The PR needs a review from the UX team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants