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

fix: enabled chats & update pawdirecte #296

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

imyanice
Copy link
Contributor

🚀 Nouvelle Pull Request

enables chats as a page, shows placeholder screen if account.service != ecoledirecte

@Louis-htmlcss
Copy link
Contributor

Ca a été fait exprès car l'interface n'est pas encore finie, à close.

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 18, 2024

Non, faut pas closed
Il faut juste changer pour que ça redirige vers la page normale des messages uniquement pour les comptes École Directe. Sinon, les autres comptes, redirige vers la page de construction

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 18, 2024

Après, à intégrer, jspas comment faire
@imyanice faut juste ajouter une condition

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 18, 2024

Genre account.service === École directe ? Placeholderscreen : Messages

@imyanice
Copy link
Contributor Author

C'est déjà dans Messages.tsx...

@imyanice
Copy link
Contributor Author

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 19, 2024

Je l'avais même pas vu dans ta pr sur école directe 😅

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

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

Dans ce cas, LGTM 👍

@imyanice
Copy link
Contributor Author

je mets à jour pawdirecte également car j'ai oublié

@imyanice imyanice changed the title fix: enabled chats fix: enabled chats & update pawdirecte Oct 19, 2024
@Louis-htmlcss
Copy link
Contributor

Non, faut pas closed
Il faut juste changer pour que ça redirige vers la page normale des messages uniquement pour les comptes École Directe. Sinon, les autres comptes, redirige vers la page de construction.

L'interface est finie pour l'école directe, j'ai rien compris.

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 19, 2024

PR #288 -> intégration messagerie École Directe

Était prévu dans la pr la page indisponible pour les autres comptes :

https://github.com/PapillonApp/Papillon/blob/main/src/views/account/Chat/Messages.tsx#L62

Voilà

@Louis-htmlcss
Copy link
Contributor

ah deso je croyais que l'interface pour les messages etait pas finit

@Kgeek33
Copy link
Contributor

Kgeek33 commented Oct 19, 2024

ah deso je croyais que l'interface pour les messages etait pas finit

C'est celle de pronote qui n'est pas encore terminée

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