Blyssco/Liber_Incantamentum
1
0
Fork 0
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

1 - Setup API Clean Architecture #13

Merged
Blyssco merged 5 commits from feature/1 into develop 2025-07-08 21:15:28 +00:00
Conversation 7 Commits 5 Files Changed 8 +106 -37
Blyssco commented 2025-07-05 14:02:23 +00:00
Owner
Copy Link
No description provided.
Blyssco added 3 commits 2025-07-05 14:02:24 +00:00
Jonathan commented 2025-07-07 06:55:08 +00:00
Owner
Copy Link

Les noms des commit sont assez minimaux.

Edit config : En quoi les config ont été modifié?
Fix : qu'est ce qui as été fixé?

De plus, regarde le principe de commit atomique. ce n'est pas obligatoire, mais c'est une tres bonne pratique.

Les noms des commit sont assez minimaux. Edit config : En quoi les config ont été modifié? Fix : qu'est ce qui as été fixé? De plus, regarde le principe de commit atomique. ce n'est pas obligatoire, mais c'est une tres bonne pratique.
Jonathan commented 2025-07-07 06:56:05 +00:00
Owner
Copy Link

Tu dois me mettre en evaluateur et en assigné sur la pr stp

Tu dois me mettre en evaluateur et en assigné sur la pr stp
Jonathan commented 2025-07-07 07:18:09 +00:00
Owner
Copy Link

Le nom de la branche feature/1 suit les precepte du gitflow, mais est tres peu pertinent

Le nom de la branche feature/1 suit les precepte du gitflow, mais est tres peu pertinent
Jonathan requested changes 2025-07-07 07:24:21 +00:00
Liber_Incantamentum.API/Class1.cs Outdated
@ -0,0 +2,4 @@
{
public class Class1
{
Jonathan commented 2025-07-07 06:57:11 +00:00
Owner
Copy Link

Une classe vide ne devrais pas éxister dans une pr, si elle est vide, c'est que soit, elle ne devrais pas se trouver dans la pr, soit la feature qu'elle doit implémenter n'exisqte pas encore.

Une classe vide ne devrais pas éxister dans une pr, si elle est vide, c'est que soit, elle ne devrais pas se trouver dans la pr, soit la feature qu'elle doit implémenter n'exisqte pas encore.
Blyssco commented 2025-07-07 18:04:06 +00:00
Author
Owner
Copy Link

Lorsque j'étais en vue solution je ne voyais pas cette classe, je vais travailler en vue folder dorénavant

Lorsque j'étais en vue solution je ne voyais pas cette classe, je vais travailler en vue folder dorénavant
Liber_Incantamentum.Infrastructure/Liber_Incantamentum.Infrastructure.csproj
@ -0,0 +7,4 @@
</PropertyGroup>
<ItemGroup>
<Folder Include="Data\" />
Jonathan commented 2025-07-07 07:22:38 +00:00
Owner
Copy Link

Pour quek raison ton infrastructure ne reference pas ton domaine? Comment comptais tu implémenter tes repos sans reference au domaine dans ton infra?

Pour quek raison ton infrastructure ne reference pas ton domaine? Comment comptais tu implémenter tes repos sans reference au domaine dans ton infra?
Blyssco commented 2025-07-07 18:04:48 +00:00
Author
Owner
Copy Link

C'est autoincrémenté dans visual studio lorsque plus tard je fais appel depuis le domaine à une classe de l'infra par exemple donc de base c'est vide

C'est autoincrémenté dans visual studio lorsque plus tard je fais appel depuis le domaine à une classe de l'infra par exemple donc de base c'est vide
Jonathan commented 2025-07-08 06:59:15 +00:00
Owner
Copy Link

OK. Donc on verra quand ce sera appeler.

OK. Donc on verra quand ce sera appeler.
Liber_Incantamentum/Properties/launchSettings.json Outdated
@ -13,7 +13,7 @@
"https": {
"commandName": "Project",
"dotnetRunMessages": true,
"launchBrowser": false,
Jonathan commented 2025-07-07 06:58:38 +00:00
Owner
Copy Link

C'est une api? Quel est l'interet d'ouvrir le browser automatiquement ?

C'est une api? Quel est l'interet d'ouvrir le browser automatiquement ?
Blyssco commented 2025-07-07 18:05:27 +00:00
Author
Owner
Copy Link

En effet mauvaise habitude, j'aime bien lancer le browser et utliser scalar mais je peux utiliser postman

En effet mauvaise habitude, j'aime bien lancer le browser et utliser scalar mais je peux utiliser postman
Jonathan commented 2025-07-08 07:00:50 +00:00
Owner
Copy Link

LGTM

LGTM
Blyssco added 1 commit 2025-07-07 18:00:07 +00:00
Blyssco added 1 commit 2025-07-07 18:02:50 +00:00
Blyssco commented 2025-07-07 18:06:40 +00:00
Author
Owner
Copy Link

Les noms des commit sont assez minimaux.

Edit config : En quoi les config ont été modifié?
Fix : qu'est ce qui as été fixé?

De plus, regarde le principe de commit atomique. ce n'est pas obligatoire, mais c'est une tres bonne pratique.

=> Ok, j'appliquerai ce principe dorénavant

Tu dois me mettre en evaluateur et en assigné sur la pr stp

=> Je ne pavenais pas à te setter, je regarderai pour la prochaine pr

Le nom de la branche feature/1 suit les precepte du gitflow, mais est tres peu pertinent

=> Je continue ainsi ?

Les noms des commit sont assez minimaux. Edit config : En quoi les config ont été modifié? Fix : qu'est ce qui as été fixé? De plus, regarde le principe de commit atomique. ce n'est pas obligatoire, mais c'est une tres bonne pratique. => Ok, j'appliquerai ce principe dorénavant Tu dois me mettre en evaluateur et en assigné sur la pr stp => Je ne pavenais pas à te setter, je regarderai pour la prochaine pr Le nom de la branche feature/1 suit les precepte du gitflow, mais est tres peu pertinent => Je continue ainsi ?
Jonathan commented 2025-07-08 07:03:19 +00:00
Owner
Copy Link

Pour. Les nom de branche, ouep, continue, juste, fait correspondre le. Nom. De. La branche avec la feature que tu implémente.

Genre, feature/kickstart-project plutôt que feature/1

Pour. Les nom de branche, ouep, continue, juste, fait correspondre le. Nom. De. La branche avec la feature que tu implémente. Genre, feature/kickstart-project plutôt que feature/1
Jonathan commented 2025-07-08 07:04:13 +00:00
Owner
Copy Link

Pour moi, c'est OK.

Je ne sais pas valider. Donc je te. Laisse merger.

Pour moi, c'est OK. Je ne sais pas valider. Donc je te. Laisse merger.
Blyssco merged commit 659d263ea7 into develop 2025-07-08 21:15:28 +00:00
Blyssco self-assigned this 2025-07-08 21:17:08 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Jonathan
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Jonathan Blyssco
No Assignees Blyssco
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Blyssco/Liber_Incantamentum#13
No description provided.
Delete Branch "feature/1"

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?