UX/UI: Ajout de la page "Présentation" de ma structure#5138
UX/UI: Ajout de la page "Présentation" de ma structure#5138hellodeloo merged 1 commit intomasterfrom
Conversation
hellodeloo
changed the title
hellodeloo
added
the
1-recette-jetable
|
J'ai ajouté cette page, ça semble ok mais j'ai besoin de professionnels de la profession car j'ai des doutes sur :
Aussi, je me demande s'il ne faudrait pas créer un nouveau test pour cette page ? Si quelqu'un est chaud, il peut le faire. |
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
28e6739 to
fb500d4
Compare
itou/www/companies_views/urls.py
Outdated
| app_name = "companies_views" | ||
|
|
||
| urlpatterns = [ | ||
| path("company_home", views.company_home, name="company_home"), |
There was a problem hiding this comment.
Les URLs et vues sont déjà dans des espaces de noms donc pas besoin de rajouter (et donc dupliquer) le "company".
Et je réserverais l'utilisation de "home" uniquement aux pages d'accueil, ça évitera des incompréhensions.
Proposition :
| path("company_home", views.company_home, name="company_home"), | |
| path("overview", views.overview, name="overview"), |
There was a problem hiding this comment.
Je n'avais rien trouvé d'autre que ce company_home. Bien mieux "overview"
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
5e724dd to
a5644fd
Compare
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
a5644fd to
b986f86
Compare
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
2 times, most recently
from
d9ddf06 to
8084025
Compare
rsebille
left a comment
rsebille
left a comment
There was a problem hiding this comment.
J'ai pas testé ma proposition pour le back_url mais ça a l'air d'être fait comme ça ailleurs 😁.
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
8084025 to
d83c994
Compare
francoisfreitag
force-pushed
the
deloo/add-main-company-page
branch
from
d83c994 to
56da2b9
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
56da2b9 to
befc192
Compare
| {% url 'companies_views:card' siae_id=company.pk as company_url %} | ||
| {% if back_url|default:'' and not open_in_tab|default:False %} | ||
| {% url_add_query company_url job_seeker=job_seeker.public_id|default:'' back_url=back_url as company_url_params %} | ||
| {% if show_cta|default_if_none:True %} |
There was a problem hiding this comment.
Après comme tu as fait l'effort d'aller le définir sur tous les include, tu pourrais même enlever le default_if_none.
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
befc192 to
96bc573
Compare
hellodeloo
force-pushed
the
deloo/add-main-company-page
branch
from
96bc573 to
c42f9e8
Compare
4 participants
🤔 Pourquoi ?
Dans le cadre de la refonte et de la navigation latérale, nous aurons un onglet “structure” qui permettra d’accéder aux différentes pages qu’on trouve actuellement dans le cadre “structure” du TDB.
Le but de cette nouvelle page est de donner une vue d’ensemble des informations de la structure.