entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

cmis: reload object after folder creation (#73794) #57

Closed
nroche wants to merge 1 commits from wip/xxxxxx-cmis-reload-folder-object into main
pull from: wip/xxxxxx-cmis-reload-folder-object
merge into: entrouvert:main
Conversation 6 Commits 1 Files Changed 2 +18 -5
nroche commented 2023-01-24 16:19:27 +01:00
Owner
Copy Link

Pour contourner un problème de configuration à Toulouse.

Pour contourner un problème de configuration à Toulouse.
nroche added 1 commit 2023-01-24 16:19:28 +01:00
gitea-wip/passerelle/pipeline/pr-main There was a failure building this commit Details
d2318231fd cmis: reload object after folder creation (#73794)
fpeters commented 2023-01-24 16:26:09 +01:00
Owner
Copy Link

Je n'ai pas envie qu'on pourrisse le connecteur pour contourner les mauvaises réponses de leur serveur.

Au très très pire, settings.MISERE et faire n'importe quoi uniquement quand c'est posé.

Je n'ai pas envie qu'on pourrisse le connecteur pour contourner les mauvaises réponses de leur serveur. Au très très pire, settings.MISERE et faire n'importe quoi uniquement quand c'est posé.
fpeters commented 2023-01-24 16:28:13 +01:00
Owner
Copy Link

Alternativement, reconnaitre que le cas de confusion de domaine est régulier, et avoir quelque chose pour "proprement" le gérer en corrigeant les réponses apportées (pas en faisant des requêtes alternatives au fur et à mesure de moments découverts).

Alternativement, reconnaitre que le cas de confusion de domaine est régulier, et avoir quelque chose pour "proprement" le gérer en corrigeant les réponses apportées (pas en faisant des requêtes alternatives au fur et à mesure de moments découverts).
bdauvergne commented 2023-01-24 16:40:06 +01:00
Owner
Copy Link

Je ne suis pas d'accord, ce serait pire d'implémenter un truc qui corrige les réponses et le projet est suffisamment en retard.

Je ne suis pas d'accord, ce serait pire d'implémenter un truc qui corrige les réponses et le projet est suffisamment en retard.
nroche closed this pull request 2023-01-24 16:43:37 +01:00
nroche commented 2023-01-24 16:45:03 +01:00
Author
Owner
Copy Link

(j'ai supprimé la branche parce qu'elle était mal nommée : wip/xxxxx-...)

(j'ai supprimé la branche parce qu'elle était mal nommée : wip/xxxxx-...)
fpeters commented 2023-01-24 16:45:09 +01:00
Owner
Copy Link

Sérieusement, plutôt que

+                    folder = self.get_object(folder.getObjectId())

avoir

+                    if settings.MISERE:
+                        # c'est cassé on contourne comme on peut on est en retard
+                        folder = self.get_object(folder.getObjectId())
Sérieusement, plutôt que ``` + folder = self.get_object(folder.getObjectId()) ``` avoir ``` + if settings.MISERE: + # c'est cassé on contourne comme on peut on est en retard + folder = self.get_object(folder.getObjectId()) ```
fpeters commented 2023-01-24 16:48:18 +01:00
Owner
Copy Link

(j'ai supprimé la branche parce qu'elle était mal nommée : wip/xxxxx-...)

Bouton "modifier" à côté du titre de la PR et là tu peux choisir une branche différente; il n'y a pas besoin de fermer.

> (j'ai supprimé la branche parce qu'elle était mal nommée : wip/xxxxx-...) Bouton "modifier" à côté du titre de la PR et là tu peux choisir une branche différente; il n'y a pas besoin de fermer.
Some checks failed
gitea-wip/passerelle/pipeline/pr-main There was a failure building this commit
Details

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
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/passerelle#57
No description provided.
Delete Branch "wip/xxxxxx-cmis-reload-folder-object"

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?