public/foundryvtt-reve-de-dragon
1
1
Fork 1
Code Issues 4 Pull Requests 1 Actions Projects Releases 166 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
Bug
Doing
Duplicated
Feature
To Do
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

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?