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

ui(landing): change cta disposition and some wording #2387

Merged
merged 4 commits into from
Oct 9, 2024

Conversation

Holist
Copy link
Collaborator

@Holist Holist commented Oct 8, 2024

close #2376
close #2354

On change les CTA en dessous du titre et en bas de la page (démo)
On cible le RSA dans la phrase d'accroche
On rajoute 'en masse' pour l'envoi des mail et SMS

Vidéo de présentation des changements

Enregistrement.de.l.ecran.2024-10-08.a.17.51.15.mp4

@Holist Holist marked this pull request as draft October 8, 2024 12:28
@Holist Holist self-assigned this Oct 8, 2024
@Holist Holist marked this pull request as ready for review October 8, 2024 15:55
@Holist Holist requested a review from aminedhobb October 8, 2024 15:55
aminedhobb
aminedhobb previously approved these changes Oct 8, 2024
Copy link
Collaborator

@aminedhobb aminedhobb left a comment

Choose a reason for hiding this comment

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

Merci pour la vidéo @Holist très cool 🙏 !

app/views/website/static_pages/welcome.html.erb Outdated Show resolved Hide resolved
@Holist Holist merged commit 5ec22c8 into staging Oct 9, 2024
7 checks passed
@Holist Holist deleted the ui/better_cta_for_landing branch October 9, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants