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/recurrent date field #1098

Closed
wants to merge 52 commits into from

Conversation

J9rem
Copy link
Contributor

@J9rem J9rem commented Sep 18, 2023

Je propose cette PR pour intégrer la possibilité d'avoir des évènements récurrents.

Contexte : cf. forum : si cette PR correspond à l'esprit YesWiki, il n'y a pas d'urgence à la relire/intégrer.

Ce que ça fait:

  • ajouter ce qu'il faut dans DateService pour créer/supprimer les fiches correspondant aux événements récurrents lors de la modification de l'évènement de tête
  • modifier le champ DateField et les templates associés pour permettre la configuration des informations pour la récurrence de l'évènement et l'affichage des ces informations lors de l'affichage de la fiche.
  • les informations de récurrence sont sauvegardées dans la fiche avec le nom de bf_date_fin_evenement_data

Comment tester:

  • créer une fiche de type évènement
  • cocher la case Évènement récurrent puis suivre les instructions

@mrflos
Copy link
Contributor

mrflos commented Sep 19, 2023

Hello,
Pour pouvoir tester il faut enlever and config.activateEventRepetition is same as true du inputs/date.twig
Sinon, je ne comprends pas l’utilité du nombre max de répétitions si l'on demande une date de fin de repetition, pourquoi on a t'on besoin? Je serai d'avis de l'enlever et de gérer avec la date de fin de repetition.

Ya un peu de taf a faire niveau ergonomie:
Je pense que la question de la répétition devrait être sur la meme ligne que la date avec un premier niveau de choix (Aucune (défaut), journalière, hebdomadaire, mensuelle, annuelle), puis a la ligne en fonction des choix "tous les" <select> "jour|semaines|mois|ans". Je pourrais regarder plus tard si tu n'as pas le temps.

Mais sinon la feature est cool!

@J9rem
Copy link
Contributor Author

J9rem commented Sep 19, 2023

Oups, bien vu, une ligne de test est restée. Je corrige de suite

@J9rem
Copy link
Contributor Author

J9rem commented Sep 19, 2023

Merci pour le compliment.

Pour le design, c'est pas trop mes compétences, ... donc effectivement, un peu d'aide maintenant (ou plus tard) me serait bienvenue.

@J9rem
Copy link
Contributor Author

J9rem commented Sep 21, 2023

@mrflos je viens de faire un premier commit pour améliorer l'affichage en essayant d'aller dans le sens de ce que tu as évoqué. J'espère que c'est plus joli/compréhensible. Je pense que c'est perfectible mais je trouve que ça commence à être bien fonctionnel.

Le deuxième commit est un refactor pour que le traitement réalisé sur les évènements pour les fiches bazar reste dans bazar en mettant ce traitement dans un service dédiée YesWiki\Bazar\Service\DateService. Le service DateService du cœur reste présent dans le dossier includes/services car il permet d'avoir à disposition un processus de mise à jour des fuseaux horaires propre à tous YesWiki et pas que bazar.
J'espère que j'ai été assez clair et que tu vois l'intérêt en termes de cloisonnement des fonctionnalités.

@J9rem
Copy link
Contributor Author

J9rem commented Oct 27, 2023

Je viens d'ajouter un commit pour gérer les dates à ne pas retenir pour la récurrence ('sauf...')

@J9rem
Copy link
Contributor Author

J9rem commented Mar 14, 2024

@Zoldiru j'ai fait un refactor de ma proposition du fichier tools/bazar/templates/fields/data.twig.
Est-ce que ça te paraît plus facile à comprendre pour le nettoyer ?
d7382ef

@J9rem J9rem force-pushed the feat/recurrent-date-field branch from 86b5c3e to a38c36c Compare April 24, 2024 10:03
@J9rem
Copy link
Contributor Author

J9rem commented Apr 24, 2024

Je retire ma PR car les derniers travaux sur cette branche ont été pilotés ou réalisés par @Zoldiru qui porte donc la #1149 en vue de l'intégration dans le coeur

@J9rem J9rem closed this Apr 24, 2024
@J9rem J9rem deleted the feat/recurrent-date-field branch May 16, 2024 17:30
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.

None yet

3 participants