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.

Sign up for GitHub

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

Jump to bottom

Evénements : pouvoir créer des widgets #837

Merged
merged 3 commits into from
May 6, 2023
Merged

Evénements : pouvoir créer des widgets #837

merged 3 commits into from
May 6, 2023

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented May 3, 2023 •
edited
Loading

Suite de #836

Quoi ?

Similaire à la création de widget pour un type de créneau donné, ici on permet de créer un widget pour un type d'événement donné.

Captures d'écran

URL Image
/event image
/event/widget_generator image

@raphodn raphodn self-assigned this May 3, 2023
@raphodn raphodn requested review from symartin and petitalb May 3, 2023 13:53
symartin
symartin approved these changes May 5, 2023
View reviewed changes
Copy link
Collaborator

@symartin symartin left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Hello,
cool, on va pouvoir les mettre sur la page d'accueil 😀.

Par contre, le truc, c'est que le bouton d’accès c'est pas très logic, pour créer un widget des job (type de créneau) c'est ds /admin, mais pour les éventement, c'est ds la sous page event/index. il faudrait peut être mettre celui des job dans admin/job/ 🤷‍♂️ ?

@symartin symartin linked an issue May 5, 2023 that may be closed by this pull request
Améliorer les événements #817
Open
@raphodn
Copy link
Member Author

raphodn commented May 6, 2023 •
edited
Loading

Par contre, le truc, c'est que le bouton d’accès c'est pas très logic, pour créer un widget des job (type de créneau) c'est ds /admin, mais pour les éventement, c'est ds la sous page event/index. il faudrait peut être mettre celui des job dans admin/job/ man_shrugging ?

c'est plutôt un widget de créneaux (/shift) que de postes (/job) non ? enfin la différence est subtile ^^
mais en effet y'a pas vraiment de raison de garder le générateur de widget à la racine de l'admin..
bon je vais peut-être modifier la PR précédente - #836 - et basculer une partie dans JobController indeed 🤔

@raphodn raphodn removed a link to an issue May 6, 2023
Améliorer les événements #817
Open
Base automatically changed from raphodn/event-widget to master May 6, 2023 15:53
@raphodn raphodn merged commit c3efd21 into master May 6, 2023
@raphodn raphodn deleted the raphodn/event-widget-2 branch May 6, 2023 16:39
@raphodn raphodn linked an issue May 6, 2023 that may be closed by this pull request
Améliorer les événements #817
Open
@raphodn raphodn mentioned this pull request May 31, 2023
Merged
@raphodn raphodn mentioned this pull request Dec 15, 2023
Open
OursDesCavernes pushed a commit to Les400Coop/gestion-compte that referenced this pull request Jan 20, 2024
@raphodn @OursDesCavernes
Evénements : pouvoir créer des widgets (elefan-grenoble#837)
84c1c83 
* Simple event widget generate page

* Widget result

* Wording changes. Query optimisations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@symartin symartin symartin approved these changes

@petitalb petitalb Awaiting requested review from petitalb

Assignees

@raphodn raphodn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Améliorer les événements
2 participants
@raphodn @symartin