entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

applification: bundle malformé (#88132) #246

Merged
lguerin merged 2 commits from wip/88132-application-errors into main 2024-03-25 10:05:32 +01:00
Conversation 2 Commits 2 Files Changed 6 +312 -133
lguerin commented 2024-03-21 15:26:42 +01:00
Owner
Copy Link

0001: le dev
0002: pour faire comme dans chrono, mais je peux faire un ticket à part si besoin

0001: le dev 0002: pour faire comme dans chrono, mais je peux faire un ticket à part si besoin
lguerin added 2 commits 2024-03-21 15:26:42 +01:00
gitea/combo/pipeline/head This commit looks good Details
e6c08a29d6
export_import: limit APIs to admin users (#88132)
lguerin changed title from WIP: applification: bundle malformé (#88132) to applification: bundle malformé (#88132) 2024-03-21 16:11:45 +01:00
pmarillonnet requested review from pmarillonnet 2024-03-21 16:54:20 +01:00
pmarillonnet commented 2024-03-21 17:08:38 +01:00
Owner
Copy Link

(en fait j’aurai pas le temps ni l’énergie de relire ça aujourd’hui, je reprends la semaine prochaine sauf si quelqu’un se dévoue pour relire entre temps 🙂)

(en fait j’aurai pas le temps ni l’énergie de relire ça aujourd’hui, je reprends la semaine prochaine sauf si quelqu’un se dévoue pour relire entre temps 🙂)
pmarillonnet refused to review 2024-03-21 17:08:43 +01:00
pmarillonnet approved these changes 2024-03-25 09:28:35 +01:00
pmarillonnet left a comment
Owner
Copy Link

Deux choses vues au passage et que je me note pour moi-même, sans pour autant que ça remette en cause la PR. Ack.

Deux choses vues au passage et que je me note pour moi-même, sans pour autant que ça remette en cause la PR. Ack.
combo/apps/export_import/api_views.py Outdated
@ -229,1 +211,3 @@
}
manifest = json.loads(tar.extractfile('manifest.json').read().decode())
except KeyError:
raise APIError(_('Invalid tar file, missing manifest'))
pmarillonnet commented 2024-03-25 09:26:36 +01:00
Owner
Copy Link

Ok ici l’ensemble de l’API est déjà écrite alors ça ne changera pas, mais je pense qu’on aurait raisonnablement pu envoyer autre chose que des http 200 sur ces erreurs de requête d’applification ratées (sans pour autant que ça prête à confusion entre l’applicatif et le technique).

Ok ici l’ensemble de l’API est déjà écrite alors ça ne changera pas, mais je pense qu’on aurait raisonnablement pu envoyer autre chose que des http 200 sur ces erreurs de requête d’applification ratées (sans pour autant que ça prête à confusion entre l’applicatif et le technique).
lguerin commented 2024-03-25 09:44:21 +01:00
Author
Owner
Copy Link

bien vu, hobo ne gère pas les 200 avec err!=0, j'ai changé ça

bien vu, hobo ne gère pas les 200 avec err!=0, j'ai changé ça
combo/apps/export_import/api_views.py Outdated
@ -230,0 +225,4 @@
if element.get('type') != page_type:
continue
try:
page = Page.objects.get(uuid=element['slug'])
pmarillonnet commented 2024-03-25 09:27:24 +01:00
Owner
Copy Link

Pareil ici, ce n’est pas la PR qui introduit ça et de toute façon l’API est déjà écrite, mais pour un relecteur peu éclairé comme moi cette confusion entre uuid et slug n’aide pas trop :/

Pareil ici, ce n’est pas la PR qui introduit ça et de toute façon l’API est déjà écrite, mais pour un relecteur peu éclairé comme moi cette confusion entre uuid et slug n’aide pas trop :/
lguerin commented 2024-03-25 09:44:03 +01:00
Author
Owner
Copy Link

yes, contrainte hobo

yes, contrainte hobo
lguerin force-pushed wip/88132-application-errors from e6c08a29d6 to 820bab39b7 2024-03-25 09:43:34 +01:00 Compare
lguerin commented 2024-03-25 09:45:56 +01:00
Author
Owner
Copy Link

(au passage, petit ajustement pour prendre en compte une remarque sur la PR analogue côté lingo, j'ai retiré l'authent dans le test test_page_redirect, la vue n'est pas décorée, puisqu'elle fait un redirect sur une vue elle-même décorée)

(au passage, petit ajustement pour prendre en compte une remarque sur la PR analogue côté lingo, j'ai retiré l'authent dans le test `test_page_redirect`, la vue n'est pas décorée, puisqu'elle fait un redirect sur une vue elle-même décorée)
lguerin merged commit 820bab39b7 into main 2024-03-25 10:05:32 +01:00
lguerin deleted branch wip/88132-application-errors 2024-03-25 10:05:32 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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/combo#246
No description provided.
Delete Branch "wip/88132-application-errors"

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?