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

Add allowedRediretionURL field to brevo config #114

Merged
merged 12 commits into from
Jan 9, 2025

Conversation

juliawegmayr
Copy link
Contributor

@juliawegmayr juliawegmayr commented Oct 23, 2024

Description

Add allowedRedirectionURL to BrevoConfigPage. Validate input in Form.

Example

[x] I have verified if my change requires an example

Screenshots/screencasts

Screenshot 2024-12-19 at 09 37 19 Screenshot 2024-12-19 at 09 37 31

Changeset

[x] I have verified if my change requires a changeset

Related tasks and documents

https://vivid-planet.atlassian.net/browse/COM-1065

@juliawegmayr juliawegmayr self-assigned this Nov 19, 2024
@juliawegmayr juliawegmayr changed the base branch from main to next December 17, 2024 08:13
@thomasdax98 thomasdax98 added this to the v3 milestone Dec 17, 2024
@juliawegmayr juliawegmayr force-pushed the add-redirection-url-field-to-brevo-config branch from a9cf950 to 4d6c71a Compare December 18, 2024 14:49
@juliawegmayr juliawegmayr marked this pull request as ready for review December 19, 2024 08:40
@auto-assign auto-assign bot requested a review from raphaelblum December 19, 2024 08:40
@juliawegmayr
Copy link
Contributor Author

juliawegmayr commented Dec 19, 2024

I adapted the information in the label of the field:

Screenshot 2024-12-19 at 11 48 36

@juliawegmayr juliawegmayr changed the title Add redirection url field to brevo config Add allowedRediretionURL field to brevo config Dec 19, 2024
@raphaelblum raphaelblum merged commit cc4bd07 into next Jan 9, 2025
1 check passed
@raphaelblum raphaelblum deleted the add-redirection-url-field-to-brevo-config branch January 9, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants