-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
Feature/upgradeapplicantstateview #1603
base: master
Are you sure you want to change the base?
Conversation
@magsyg Jeg har en PR på akkurat dette |
backend/root/utils/routes.py
Outdated
@@ -564,6 +564,7 @@ | |||
samfundet__interview_list = 'samfundet:interview-list' | |||
samfundet__interview_detail = 'samfundet:interview-detail' | |||
samfundet__api_root = 'samfundet:api-root' | |||
samfundet__api_root = 'samfundet:api-root' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fjern linje
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linjen er der fortsatt
backend/samfundet/urls.py
Outdated
@@ -105,6 +105,11 @@ | |||
views.RecruitmentApplicationForGangUpdateStateView.as_view(), | |||
name='recruitment_application_update_state_gang', | |||
), | |||
path( | |||
'recruitment-position-organized-applications/<slug:pk>/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hvorfor slug
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Er det som er gjort andre steder
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Slug er litt misvisende syns jeg, her er det jo en ID som brukes. Slug er mer for strings som er gjort URL-vennlig.
For eksempel: https://stackoverflow.com/questions/4230846/what-is-the-etymology-of-slug-in-a-url
Her er what-is-the-etymology-of-slug-in-a-url
slugen, som er generert fra tittelen What is the etymology of 'slug' in a URL?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oki byttet det til id
a68d435
to
c4abc08
Compare
Make a view for and serializer where applicants are organized
Before it was done in frontend, now it is moved to backend
This will also make it easier to fetch the status with how dynamic it is