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.

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

Recruitmentvalidation and Form Errors #877

Merged
merged 19 commits into from
Feb 15, 2024
Merged

Conversation

magsyg
Copy link
Contributor

@magsyg magsyg commented Dec 21, 2023

La til validerings metode som kanskje kan standardiseres gjennom resten av serializerne, returnererer field errors som json istedenfor å krasje fra clean metoden til et object

Samtidig la til muligheten til å vise errors from backend i frontend
For er jævlig piss å ikke få bekreftelse av feilene, og kun en error melding

bilde

@magsyg magsyg force-pushed the feature/recruitmentvalidation branch from c8bd161 to 1fd30a0 Compare December 21, 2023 16:37
@magsyg magsyg force-pushed the feature/recruitmentvalidation branch from 1fd30a0 to cff4bd5 Compare December 21, 2023 16:42
@magsyg magsyg self-assigned this Dec 21, 2023
Snorre98
Snorre98 previously approved these changes Jan 5, 2024
Copy link
Contributor

@Snorre98 Snorre98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blir sikkert bra når convos er resolved :)

@magsyg
Copy link
Contributor Author

magsyg commented Jan 6, 2024

@emilte dis gud?

@magsyg magsyg force-pushed the feature/recruitmentvalidation branch from e95dd91 to 5dcd659 Compare January 6, 2024 20:35
@magsyg magsyg force-pushed the feature/recruitmentvalidation branch from 8743fd6 to f70bc59 Compare February 15, 2024 18:08
backend/samfundet/views.py Outdated Show resolved Hide resolved
@magsyg magsyg merged commit 32340c6 into master Feb 15, 2024
5 checks passed
@magsyg magsyg deleted the feature/recruitmentvalidation branch February 15, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants