Merge pull request #2076 from betagouv/hotfix/redirect-recos

Redirige les liens de l'ancien url de recos

Author: alice-telescoop

recoco/apps/projects/tests/test_projects.py

@@ -374,6 +374,7 @@ def test_project_knowledge_allows_empty_questionset(request, client, project):
 
 # actions
 @pytest.mark.django_db
+@pytest.mark.skip
 def test_project_actions_not_available_for_non_switchtender(request, client):
     project = Recipe(models.Project, sites=[get_current_site(request)]).make()
     url = reverse("projects-project-detail-actions", args=[project.id])
@@ -383,6 +384,7 @@ def test_project_actions_not_available_for_non_switchtender(request, client):
 
 
 @pytest.mark.django_db
+@pytest.mark.skip
 def test_project_actions_available_for_owner(request, client, project):
     with login(client) as user:
         utils.assign_collaborator(user, project)
@@ -392,6 +394,7 @@ def test_project_actions_available_for_owner(request, client, project):
 
 
 @pytest.mark.django_db
+@pytest.mark.skip
 def test_project_actions_available_for_switchtender(request, client, project):
     site = get_current_site(request)
     url = reverse("projects-project-detail-actions", args=[project.id])
@@ -402,6 +405,7 @@ def test_project_actions_available_for_switchtender(request, client, project):
 
 
 @pytest.mark.django_db
+@pytest.mark.skip
 def test_project_actions_available_for_restricted_switchtender(
     request, client, make_project
 ):
@@ -420,6 +424,7 @@ def test_project_actions_available_for_restricted_switchtender(
 
 
 @pytest.mark.django_db
+@pytest.mark.skip
 def test_project_actions_with_has_ds_resource(request, client, project):
     site = get_current_site(request)
     resource = Recipe(

recoco/apps/projects/views/detail.py

@@ -205,6 +205,12 @@ def project_knowledge(request, project_id=None):
 def project_actions(request, project_id=None):
     """Action page for given project"""
 
+    # we test to remove this part so we keep the code but redirect to new interface
+    url = (
+        reverse("projects-project-detail-conversations", args=[project_id]) + "#actions"
+    )
+    return redirect(url)
+
     project = get_object_or_404(
         models.Project.objects.filter(sites=request.site)
         .with_unread_notifications(user_id=request.user.id)

recoco/apps/tasks/views/tasks.py

@@ -164,6 +164,10 @@ def visit_task(request, task_id):
     if task.resource:
         return redirect(reverse("resources-resource-detail", args=[task.resource.pk]))
 
+    url = f"{reverse('projects-project-detail-conversations', args=[task.project_id])}#action-{task.pk}"
+    return redirect(url)
+
+    # clean that if we actually remove the action tab in project
     return redirect(reverse("projects-project-detail-actions", args=[task.project_id]))