Skip to content

298 confirm button behind email links#16

Merged
tijsverkoyen merged 4 commits intomainfrom
298-confirm-button-behind-email-links
Mar 26, 2024
Merged

298 confirm button behind email links#16
tijsverkoyen merged 4 commits intomainfrom
298-confirm-button-behind-email-links

Conversation

@bjorvack
Copy link
Copy Markdown
Contributor

@bjorvack bjorvack requested a review from a team March 26, 2024 08:43
@tijsverkoyen
Copy link
Copy Markdown
Member

Too many commits? Or just a wrong PR description?

Comment thread src/Controller/User/ConfirmController.php Outdated
Comment thread translations/messages+intl-icu.nl.yaml Outdated
Comment thread translations/messages+intl-icu.nl.yaml Outdated
Comment thread translations/messages+intl-icu.nl.yaml Outdated
Comment thread src/Controller/User/Ajax/PasswordStrengthController.php Outdated
{% extends '@SumoCodersFrameworkCore/user.html.twig' %}

{% block main %}
<p>{{ 'Confirm your account to continue.'|trans }}</p>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Kan je eens screenshot van deze pagina maken aub. Want kan me niet voorstellen hoe die er nu uit ziet.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@tijsverkoyen Dit is een voorbeeld voor bv dpg media
Screenshot 2024-03-26 at 11 14 40

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Die Confirm mag nog vertaald worden naar "Bevestig je account"

Copy link
Copy Markdown
Contributor Author

@bjorvack bjorvack Mar 26, 2024

Choose a reason for hiding this comment

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

fixed c1221fd

@bjorvack bjorvack force-pushed the 298-confirm-button-behind-email-links branch from 64e30cf to b062e60 Compare March 26, 2024 08:58
@tijsverkoyen tijsverkoyen merged commit ce299bb into main Mar 26, 2024
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.

2 participants