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

filr_rest: accept empty emails in share-folder (#74481) #108

Closed
Ghost wants to merge 0 commits from wip/74481-filr-accept-empty-emails-in-share-folder into main
pull from: wip/74481-filr-accept-empty-emails-in-share-folder
merge into: entrouvert:main
Conversation 2 Commits - Files Changed -
Ghost commented 2023-02-14 12:37:02 +01:00
First-time contributor
Copy Link

Noter que j'ai retiré le « 'format': 'email' » dans le schéma pour laisser passer une chaîne vide.

(Ceci étant j'ai testé d'abord en laissant de « 'format': 'email' » et ça passait aussi, y compris "not a mail", ce qui me fait dire que ce format ne vérifie rien, alors autant le retirer pour ne pas avoir de pépin le jour où il va se mettre à fonctionner)

Noter que j'ai retiré le « 'format': 'email' » dans le schéma pour laisser passer une chaîne vide. (Ceci étant j'ai testé d'abord en laissant de « 'format': 'email' » et ça passait aussi, y compris "not a mail", ce qui me fait dire que ce format ne vérifie rien, alors autant le retirer pour ne pas avoir de pépin le jour où il va se mettre à fonctionner)
ecazenave commented 2023-02-14 12:45:52 +01:00
Owner
Copy Link

On lancerait pas une erreur si aucun appel a été fait parce que aucune adresse valide ?

J'imagine que ça pourrait nous éviter des tickets de supports pénibles.

On lancerait pas une erreur si aucun appel a été fait parce que aucune adresse valide ? J'imagine que ça pourrait nous éviter des tickets de supports pénibles.
Ghost force-pushed wip/74481-filr-accept-empty-emails-in-share-folder from 5bc64d985b to 00804299d9 2023-02-14 13:02:04 +01:00 Compare
Ghost commented 2023-02-14 13:02:42 +01:00
Author
First-time contributor
Copy Link

On lancerait pas une erreur si aucun appel a été fait parce que aucune adresse valide ?
J'imagine que ça pourrait nous éviter des tickets de supports pénibles.

Bonne idée, c'est fait dans cette nouvelle version.

> On lancerait pas une erreur si aucun appel a été fait parce que aucune adresse valide ? > J'imagine que ça pourrait nous éviter des tickets de supports pénibles. Bonne idée, c'est fait dans cette nouvelle version.
Ghost requested review from ecazenave 2023-02-14 13:03:10 +01:00
ecazenave approved these changes 2023-02-14 14:13:34 +01:00
Ghost closed this pull request 2023-02-14 14:48:58 +01:00
Some checks reported errors
gitea-wip/passerelle/pipeline/pr-main This commit looks good
Details
gitea/passerelle/pipeline/head Something is wrong with the build of this commit
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
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/passerelle#108
No description provided.
Delete Branch "wip/74481-filr-accept-empty-emails-in-share-folder"

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?