entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 8 Releases Wiki Activity

general: add journal app (#86632) #236

Open
fpeters wants to merge 1 commits from wip/86632-journal into main
pull from: wip/86632-journal
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 19 +914 -19
fpeters commented 2024-03-26 10:33:05 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2024-03-26 10:33:05 +01:00
gitea/chrono/pipeline/head There was a failure building this commit Details
639833c7fe general: add journal app (#86632)
fpeters force-pushed wip/86632-journal from 639833c7fe to c8aea5747e 2024-03-26 10:49:32 +01:00 Compare
fpeters force-pushed wip/86632-journal from c8aea5747e to 57e26d83f9 2024-04-09 13:59:19 +02:00 Compare
fpeters force-pushed wip/86632-journal from 57e26d83f9 to e1aad892e1 2024-04-09 14:19:50 +02:00 Compare
fpeters force-pushed wip/86632-journal from e1aad892e1 to d87a948548 2024-04-09 14:27:35 +02:00 Compare
fpeters force-pushed wip/86632-journal from d87a948548 to c230014176 2024-04-09 14:38:52 +02:00 Compare
fpeters force-pushed wip/86632-journal from c230014176 to 6b3f37321c 2024-04-15 16:10:05 +02:00 Compare
fpeters force-pushed wip/86632-journal from 6b3f37321c to 22eae6fb48 2024-04-16 12:35:02 +02:00 Compare
fpeters changed title from WIP: general: add journal app (#86632) to general: add journal app (#86632) 2024-04-16 13:39:37 +02:00
fpeters commented 2024-04-16 13:45:47 +02:00
Author
Owner
Copy Link

Je pense avoir couvert tout ce qui était mentionné dans le ticket et que j'ai trouvé, mais de toute façon ça n'est pas compliqué ensuite d'ajouter des logs à d'autres endroits.

À venir, il y aurait je pense besoin d'une réflexion sur l'attribution des actions quand il s'agit d'appels aux API, le module qui fait appelle l'API devrait pouvoir renseigner au nom de qui il le fait; a priori via hobo/rest_authentication.py il suffit de faire ?NameID=xxx et ça fait le job, mais presque sûr qu'on ne pose pas ça systématiquement.

Je pense avoir couvert tout ce qui était mentionné dans le ticket et que j'ai trouvé, mais de toute façon ça n'est pas compliqué ensuite d'ajouter des logs à d'autres endroits. À venir, il y aurait je pense besoin d'une réflexion sur l'attribution des actions quand il s'agit d'appels aux API, le module qui fait appelle l'API devrait pouvoir renseigner au nom de qui il le fait; a priori via hobo/rest_authentication.py il suffit de faire ?NameID=xxx et ça fait le job, mais presque sûr qu'on ne pose pas ça systématiquement.
fpeters force-pushed wip/86632-journal from 22eae6fb48 to be08d26223 2024-04-22 13:01:37 +02:00 Compare
fpeters force-pushed wip/86632-journal from be08d26223 to 10074de603 2024-04-22 13:08:42 +02:00 Compare
fpeters force-pushed wip/86632-journal from 10074de603 to e2343d0fe8 2024-04-23 14:21:38 +02:00 Compare
fpeters force-pushed wip/86632-journal from e2343d0fe8 to 34586959a3 2024-04-23 14:31:56 +02:00 Compare
fpeters force-pushed wip/86632-journal from 34586959a3 to 125f1bc797 2024-04-23 14:52:23 +02:00 Compare
fpeters force-pushed wip/86632-journal from 125f1bc797 to e847739a56 2024-04-23 15:43:27 +02:00 Compare
fpeters force-pushed wip/86632-journal from e847739a56 to 22f981d83e 2024-04-23 16:05:54 +02:00 Compare
fpeters force-pushed wip/86632-journal from 22f981d83e to a1d73bb8cf 2024-04-23 16:14:09 +02:00 Compare
fpeters force-pushed wip/86632-journal from a1d73bb8cf to 95b34eb9d7 2024-04-23 17:03:15 +02:00 Compare
lguerin approved these changes 2024-04-26 11:42:05 +02:00
All checks were successful
gitea/chrono/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/86632-journal main
git pull origin wip/86632-journal

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/86632-journal
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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: entrouvert/chrono#236
No description provided.
Delete Branch "wip/86632-journal"

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?