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

#326 - Abonnement au calendrier Robert2 depuis application de calendrier externe #330

Merged
merged 20 commits into from
Mar 5, 2022

Conversation

Donov4n
Copy link
Member

@Donov4n Donov4n commented Feb 27, 2022

Fermera aussi le ticket de fin de support PHP < 7.4 (#327).
(Il faudra juste mette à jour la doc du site comme dit dans le ticket @polosson)

Copy link
Member

@polosson polosson left a comment

Choose a reason for hiding this comment

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

Yeah !
À part quelques remarques d'ordre rédactionnelle (texte dans l'UI), et quelques questionnements, c'est impeccable @Donov4n !! 👍

client/src/locale/en/common.js Outdated Show resolved Hide resolved
client/src/locale/fr/pages.js Outdated Show resolved Hide resolved
client/src/locale/fr/pages.js Outdated Show resolved Hide resolved
client/src/locale/fr/common.js Outdated Show resolved Hide resolved
client/src/locale/fr/common.js Outdated Show resolved Hide resolved
client/src/pages/User/index.js Show resolved Hide resolved
server/src/App/App.php Show resolved Hide resolved
server/tests/models/SettingTest.php Outdated Show resolved Hide resolved
@Donov4n
Copy link
Member Author

Donov4n commented Mar 5, 2022

@polosson: Tu m'as fait remplacer partout re-générer par "régénéré" mais à mon avis ça n'a pad du tout le même sens.
Je suis de l'avis suivant de la réponse de "PhL" ici: https://www.question-orthographe.fr/question/regenerer-ou-regenerer/

Là ça fait assez bizarre maintenant ...

Le vais mettre les modifs suggérée "Re-générer" => "Régénérer" dans un commit séparé histoire qu'on puisse le virer une fois que tu auras vu ce massage, si jamais on part sur l'idée de le virer.

@Donov4n Donov4n force-pushed the feature/public-calendar branch 2 times, most recently from 6225d96 to 7c8ee05 Compare March 5, 2022 09:58
@Donov4n Donov4n force-pushed the feature/public-calendar branch from 7c8ee05 to bd3d929 Compare March 5, 2022 10:01
@Donov4n
Copy link
Member Author

Donov4n commented Mar 5, 2022

Je ne sais pas pourquoi je ne peux plus répondre directement sous tes messages donc je le fais ici:

Ah, il me semblait qu'on laissait tomber cette histoire de resource provenant des metaData de la route... Mais j'ai peut-être mal compris ?

Non tu as raison, mais là c'est pas le point. Je vire la custom route qui n'a pas de sens users/signup et j'utilise les metaData comme c'est fait ailleurs dans le même component pour que ça reste homogène.
À terme il faudra migrer tout le component pour qu'il utilise un stores/api/users.ts mais ça rentre pas dans le cadre de cette PR.

@Donov4n Donov4n merged commit fec8e17 into develop Mar 5, 2022
@Donov4n Donov4n deleted the feature/public-calendar branch March 5, 2022 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants