Skip to content

Conversation

@AoifeHughes
Copy link

This is currently draft still and needs a lot of work

@AoifeHughes AoifeHughes self-assigned this Mar 4, 2024
@AoifeHughes
Copy link
Author

  • need to add unit testing for it too once it's implemented
  • implement for own users
  • implement for managers

@AoifeHughes AoifeHughes added enhancement New feature or request frontend labels Mar 4, 2024
@AoifeHughes
Copy link
Author

I've also added a scripts folder to make quick dev easier

@mhauru
Copy link
Collaborator

mhauru commented Mar 4, 2024

You can ignore for now the question of admins changing other people's passwords, because we don't have admin users at all yet.

Also, we just merged a couple of big PRs that change the backend API implementation a lot (though not the API itself). That is to say there will be a substantial merge conflict on dtbase/backend/api/user/routes.py. I can sort it out though. Would it be better to do it now, rather than at the end, or would that mess with you working on this?

@AoifeHughes
Copy link
Author

I can sort out any merge conflicts, is there anything else currently being worked on which might be affected? Happy to take that into consideration.

@mhauru
Copy link
Collaborator

mhauru commented Mar 7, 2024

All the big PRs that restructured things have now been merged. There's currently one in the works that will enforce linting and formatting of Typescript and maybe of CSS (#103), but that probably won't cause significant conflicts. I'm currently working on documentation and @EdwinB12 is on Docker compose stuff for local deployment, which sounds conflict free.

@mhauru
Copy link
Collaborator

mhauru commented Apr 12, 2024

This makes a start on addressing #141. We are all now moving on to different projects and won't have time to finish this, but I'm leaving it open as a draft in case someone comes back and wants to finish the work. Note that there's a massive merge conflict coming up on the backend side. However, the frontend HTML template file is still current and useful.

@mhauru mhauru marked this pull request as draft April 12, 2024 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants