Skip to content

Feat/login page#6

Merged
Yoctoboy merged 2 commits intomainfrom
feat/login-page
Jul 7, 2025
Merged

Feat/login page#6
Yoctoboy merged 2 commits intomainfrom
feat/login-page

Conversation

@Yoctoboy
Copy link
Collaborator

@Yoctoboy Yoctoboy commented Jun 22, 2025

  • Basic topbar with Pyronear logo (is it the right one?)
  • Simple login page (design only)
  • Language switcher with fr/en locales

image

@Yoctoboy Yoctoboy self-assigned this Jun 22, 2025
@vercel
Copy link

vercel bot commented Jun 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
new-pyro-platform ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 6, 2025 11:48am

Copy link
Collaborator

@SebastienTouze SebastienTouze left a comment

Choose a reason for hiding this comment

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

seens good =)

@m-julio
Copy link
Collaborator

m-julio commented Jul 5, 2025

@Yoctoboy Mes retours sont plutôt globaux car le code me semble clean :

  • Je pense que le logo est plutôt celui ci : (en réponse à ta question dans la PR) voir le lien https://github.com/pyronear/pyronear.github.io/tree/master/img
    logo_letters_orange
    Il me parait plus safe pour le moment de conserver le logo actuel ci-joint car si on choisit de faire évoluer le logo, il faudra le faire évoluer partout, notamment dans les supports de comm'. Ce qui me parait trop impactant en l'état.
    Eventuellement on peut ouvrir un sujet de discussion avec l'ensemble de l'asso pour évoquer le sujet
  • Actuellement il existe également la langue Espagnol (dans le front en prod) + ce sont les drapeaux qui sont affichés au lieu du texte "Français", "English", etc : il ne parait pas nécessaire de l'ajouter dans cette PR mais dans ce cas, ça te convient d'ouvrir un issue pour ne pas l'oublier ?
  • Je ne sais pas ce qui a été décidé sur les tests mais il semble qu'il y en ait un sur le composant App.tsx, je m'attends donc à en avoir un a minima pour chaque nouveau composant react ( Topbar, LanguageSwitcher et LoginForm). Qu'est ce qui a été décidé sur le sujet ?

@Yoctoboy Yoctoboy linked an issue Jul 6, 2025 that may be closed by this pull request
@Yoctoboy Yoctoboy removed this from New Platform Jul 6, 2025
@Yoctoboy
Copy link
Collaborator Author

Yoctoboy commented Jul 6, 2025

@m-julio

  • logo : tu as raison, changé
  • espagnol : ticket créé dans le projet ici
  • testing : L'avis d'FG est ici sur slack. J'ai ajouté un test basique sur LoginForm

Copy link
Collaborator

@m-julio m-julio left a comment

Choose a reason for hiding this comment

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

Ok pour moi

@Yoctoboy Yoctoboy merged commit 4e08690 into main Jul 7, 2025
3 checks passed
@Yoctoboy Yoctoboy deleted the feat/login-page branch July 7, 2025 08:17
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.

As a user, I can see the Login page

3 participants