Skip to content

chore: remove references of legacy brand users and consent#308

Merged
artur-nogueira merged 1 commit into
masterfrom
chore/FT-1105-remove-brand-user-consent
Apr 17, 2026
Merged

chore: remove references of legacy brand users and consent#308
artur-nogueira merged 1 commit into
masterfrom
chore/FT-1105-remove-brand-user-consent

Conversation

@artur-nogueira
Copy link
Copy Markdown
Contributor

Steps to test

  • remove references of legacy brand users and consent

Ticket

Copilot AI review requested due to automatic review settings April 17, 2026 10:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Select documentation to remove legacy references to Brand Users and Brand Consent, aligning the docs with the current Brand model and webhook behavior.

Changes:

  • Removes the Brand Consent section from select/brands.md.
  • Removes the brand.consent event payload example section from select/webhooks.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
select/webhooks.md Removes the “Brand” event payload example section from the webhook event docs.
select/brands.md Simplifies the Brand overview by removing Brand Consent-related guidance and endpoints.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread select/webhooks.md
@artur-nogueira artur-nogueira force-pushed the chore/FT-1105-remove-brand-user-consent branch from f1f408c to 9a4c1ac Compare April 17, 2026 10:45
pedroalmeidaes
pedroalmeidaes previously approved these changes Apr 17, 2026
@artur-nogueira artur-nogueira merged commit ee020ae into master Apr 17, 2026
3 checks passed
@artur-nogueira artur-nogueira deleted the chore/FT-1105-remove-brand-user-consent branch April 17, 2026 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants