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

feat(front): update website with new mockup #19

Merged
merged 8 commits into from
Aug 19, 2024

Conversation

rebeccadumazert
Copy link
Contributor

@rebeccadumazert rebeccadumazert commented Aug 13, 2024

@douglasduteil
Copy link
Member

@douglasduteil
Copy link
Member

@rebeccadumazert Is it really a modal ?
It feels like a separate page to me.

@douglasduteil
Copy link
Member

[Sry miss laptop click 🫢 ]

@rebeccadumazert rebeccadumazert force-pushed the RDUM-update-website branch 2 times, most recently from b021bc1 to 9437793 Compare August 14, 2024 16:39
</div>
</div>
</div>
</div>
{/* <Element name="componentStart">
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce qu'on peut supprimer tous les commentaires dans ce composant ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'est une PR que je vais faire après avoir mergée celle-ci. Il y a plein plein de commentaires dans le code qui datent de la première version proconnect. J'attends de mergée cette PR pour tout cleaner, je voulais pas mélanger ces deux tâches qui sont distinctes l'une de l'autre, si c'est ok pour toi

@@ -1,4 +1,4 @@
import PrivateOrPublic from '../components/PrivateOrPublic/PrivateOrPublic';
import PrivateOrPublic from '../components/HowDoesItWork/HowDoesItWork';
Copy link
Contributor

Choose a reason for hiding this comment

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

Faut renommer l'import du composant ici

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'était carrément un composant qu'il fallait supprimer, je l'ai supprimé 🙏🏻

@rebeccadumazert rebeccadumazert merged commit c00214d into main Aug 19, 2024
2 checks passed
@rebeccadumazert rebeccadumazert deleted the RDUM-update-website branch August 19, 2024 13:47
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.

3 participants