public/foundryvtt-reve-de-dragon
1
1
Fork 1
Code Issues 4 Pull Requests Actions Projects Releases 187 Wiki Activity

Fix: création calendrier en double #485

Closed
vincent.vandeme wants to merge 1 commits from v1.4-calendrier into v1.4
pull from: v1.4-calendrier
merge into: public:v1.4
Conversation 10 Commits 1 Files Changed 2 +37 -38
vincent.vandeme commented 2021-06-05 20:27:50 +02:00 (Migrated from gitlab.com)
Copy Link
  • suppression des messages lors de l'initialisation (nouveau monde)
  • await pour éviter les appels paralleles à l'initialisation
    (init et render)
- suppression des messages lors de l'initialisation (nouveau monde) - await pour éviter les appels paralleles à l'initialisation (init et render)
vincent.vandeme commented 2021-06-05 20:33:39 +02:00 (Migrated from gitlab.com)
Copy Link

await incorrect car updateDisplay n'est pas async

await incorrect car updateDisplay n'est pas async
vincent.vandeme commented 2021-06-05 20:33:40 +02:00 (Migrated from gitlab.com)
Copy Link

incrementerJour n'est pas async

incrementerJour n'est pas async
vincent.vandeme commented 2021-06-05 20:33:40 +02:00 (Migrated from gitlab.com)
Copy Link

super.getData() retourne un objet vide {}, simplification

super.getData() retourne un objet vide {}, simplification
vincent.vandeme commented 2021-06-05 20:33:40 +02:00 (Migrated from gitlab.com)
Copy Link

pas de raison de duplicate: c'est un toujours nouvel objet

pas de raison de duplicate: c'est un toujours nouvel objet
vincent.vandeme commented 2021-06-05 20:33:41 +02:00 (Migrated from gitlab.com)
Copy Link

on peut avoir des lambda async

on peut avoir des lambda async
vincent.vandeme commented 2021-06-05 20:33:41 +02:00 (Migrated from gitlab.com)
Copy Link

Le fix proprement dit: await sur initCalendrier pour que le render ligne 226 ne fasse pas un deuxième appel

Le fix proprement dit: await sur initCalendrier pour que le render ligne 226 ne fasse pas un deuxième appel
vincent.vandeme commented 2021-06-05 20:33:41 +02:00 (Migrated from gitlab.com)
Copy Link

comme formData était toujours {}, autant faire une méthode factory et éliminer ce paramètre

comme formData était toujours {}, autant faire une méthode factory et éliminer ce paramètre
vincent.vandeme commented 2021-06-05 20:33:41 +02:00 (Migrated from gitlab.com)
Copy Link

ajout d'une option pour afficher (par défaut)

ajout d'une option pour afficher (par défaut)
vincent.vandeme commented 2021-06-05 20:33:41 +02:00 (Migrated from gitlab.com)
Copy Link

Ne pas afficher les messages dans le tchat lors de l'initialisation (ie: pour un nouveau monde)

Ne pas afficher les messages dans le tchat lors de l'initialisation (ie: pour un nouveau monde)
LeRatierBretonnien commented 2021-06-05 20:42:35 +02:00 (Migrated from gitlab.com)
Copy Link

Désolé, trop de changements. Je n'applique pas.

Désolé, trop de changements. Je n'applique pas.

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Bug
Doing
Duplicated
Feature
To Do
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
VincentVk uberwald
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: public/foundryvtt-reve-de-dragon#485
No description provided.
Delete Branch "v1.4-calendrier"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?