Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ajoute un système de liens de partage #6134

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Ajoute un système de liens de partage #6134

wants to merge 1 commit into from

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Jun 10, 2021 •
edited
Loading

Fix #6092.

Il y a deux gros morceaux à faire :

  1. La gestion des liens de partage (création, modification, désactivation, réactivation, suppression, expiration).
  2. La gestion de l'accès au contenu depuis les liens.

C'est presque fini !

À faire

  • back de la gestion des liens (création, modification, désactivation, ...)
  • front de la gestion des liens
  • gestion de l'accès au contenu depuis les liens
  • écrire plus de tests

Contrôle qualité

En tant qu'auteur, tester la gestion des liens :

  • création de lien
  • modification de lien (description)
  • modification de lien (date)
  • modification de lien (type)
  • désactivation / réactivation
  • suppression

En tant que déconnecté, tester les accès aux liens :

  • pour un lien actif, dernier brouillon non expiré (attendu : on voit la page)
  • pour un lien actif, dernier brouillon expiré (attendu : 403)
  • pour un lien actif, dernière bêta avec bêta acive (attendu : on voit la page)
  • pour un lien actif, dernière bêta, avec bêta désactivée (attendu : 403)
  • pour un lien inactif, peu importe son expiration, son type, etc. (attendu : 403)
  • pour un lien supprimé (attendu : 404).

Tester la suppression en cascade : quand on supprime un contenu, les liens affairant son supprimés également.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Jun 10, 2021
@coveralls
Copy link

coveralls commented Jun 13, 2021 •
edited
Loading

Coverage Status

Coverage increased (+0.08%) to 86.772% when pulling f5a9952 on Arnaud-D:passe_moi_le_sel_pardon_le_lien into 836d6b8 on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 5befa37 to f5a9952 Compare June 14, 2021 10:02
@Arnaud-D
Copy link
Contributor Author

J'avance petit à petit. Les templates ne facilitent pas la tâche, parce qu'ils gèrent au cas par cas, la bêta, la version en ligne, le draft, etc. Il faudrait probablement restructurer un peu ça. J'hésite.

@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from e48def2 to d872644 Compare September 11, 2021 19:48
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from d872644 to a6221d9 Compare November 2, 2021 19:51
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from a6221d9 to 0873fa5 Compare June 26, 2022 17:10
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 0873fa5 to a0fe408 Compare August 14, 2022 11:45
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from a0fe408 to d8fb8ab Compare October 7, 2022 19:59
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from d8fb8ab to 4d8e28a Compare December 25, 2022 13:45
@Arnaud-D Arnaud-D mentioned this pull request Jan 26, 2023
Merged
@Situphen Situphen added the En attente de la dépendance Ticket bloqué en attente d'un correctif dans une dépendance utilisée label May 23, 2023
@Arnaud-D
Copy link
Contributor Author

En attente de #6457.

@Arnaud-D
Copy link
Contributor Author

Ça va pouvoir reprendre ici puisqu'on a fusionné #6457 !

@Arnaud-D Arnaud-D removed the En attente de la dépendance Ticket bloqué en attente d'un correctif dans une dépendance utilisée label Oct 28, 2023
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 5a1f5d3 to 9ce0191 Compare February 24, 2024 07:12
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch 2 times, most recently from 8f56aff to de32df9 Compare March 2, 2024 18:06
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from de32df9 to 62f361b Compare March 16, 2024 08:43
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 62f361b to 04665b2 Compare March 29, 2024 22:12
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 04665b2 to c5fa49b Compare April 28, 2024 06:03
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from c5fa49b to 6f67e11 Compare June 1, 2024 15:10
@Arnaud-D Arnaud-D self-assigned this Jun 9, 2024
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 6f67e11 to 612440e Compare October 18, 2024 20:09
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 612440e to 38e2195 Compare November 9, 2024 09:05
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Nov 9, 2024

J'ai repris, et j'ai réussi très rapidement à passer les obstacles qui me bloquaient avant la PR que je mentionnais auparavant.

Il reste pas mal de finitions, mais il n'y plus d'obstacle majeur.

@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch 2 times, most recently from 270b756 to 1ebd9dc Compare November 10, 2024 17:06
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from 1ebd9dc to 0c025d4 Compare November 10, 2024 18:19
@coveralls
Copy link

coveralls commented Nov 10, 2024 •
edited
Loading

Coverage Status

coverage: 89.211% (+0.01%) from 89.2%
when pulling 5e51024 on Arnaud-D:passe_moi_le_sel_pardon_le_lien
into d38b46a on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch 3 times, most recently from 5a6e4ce to 803f279 Compare November 11, 2024 17:07
@Arnaud-D
Copy link
Contributor Author

J'ai bien avancé, plus qu'à écrire plus de tests !

@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch 4 times, most recently from b06ef6b to d6fa7dc Compare December 13, 2024 08:47
@Arnaud-D Arnaud-D marked this pull request as ready for review December 13, 2024 11:08
@Arnaud-D Arnaud-D force-pushed the passe_moi_le_sel_pardon_le_lien branch from d6fa7dc to 5e51024 Compare December 14, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Arnaud-D Arnaud-D

Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Projects
Suivi des PR
Status: En attente de QA
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pouvoir partager la bêta ou un brouillon à des non-inscrits
3 participants
@Arnaud-D @coveralls @Situphen