-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
[Counties] Add or edit counties #216
Comments
Oh since I am doing the list view, you might as well add me to this too. |
@aniri sorry I missed hacktoberfest on this one however, I was trying to finish it up today. I'll let you know if I run into any issues. |
Is this up for grabs, or is @Botosio still working on it ? |
Oh sorry this was assigned last year and I must not have finished it. Please take it for this year Hacktoberfest! |
Oh ok. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add the option of adding new counties and editing of existing ones.
The add option should only be available for master admins (organizer flag set to true).
The edit option should only be available for master admins (organizer flag set to true).
Other admins will only have a view option that opens the edit county details page in view mode.
Backend part was implemented in code4romania/monitorizare-vot#254 . The endpoint for editing counties is
POST api/v1/county/{id}
UI available here: https://www.figma.com/file/61NHR7fwmV1Dsv4Pb6F8T7/code4ro_mv_user-management_2018.04.21-(Copy)?node-id=754%3A0
Counties data model:
Please use bootstrap classes as much as possible and avoid adding custom css styles to keep the codebase clean & consistent. The new UI should be as similar as possible to the one in figma, not pixel perfect.
Also, make sure all texts are extracted in translation files.
The text was updated successfully, but these errors were encountered: