Skip to content

Setup models - #5

Merged
Precipicee merged 9 commits into
use_uvfrom
setup_models
Dec 11, 2025
Merged

Setup models#5
Precipicee merged 9 commits into
use_uvfrom
setup_models

Conversation

@Archer-Key

Copy link
Copy Markdown
Collaborator

Change Summary

  • Added Task, Topic, and Time models
  • Setup Admin page to allow admin users to create entries for the above 3 models
  • Setup API pages for each of Task, Time, and Topic
  • Setup nested relations within Task API page to display Topics and Times related to the Task in the API response

Other Notes

  • This PR is not related to any issue as the issue was not created
  • This PR is intended to allow work on the individual pages and so only provides a base implementation, to be expanded later

@Precipicee

Precipicee commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator

Without having tested the code, it looks good upon reading and I'd be happy to merge it, as you've said it all works and it aligns with all the stuff we've discussed as a group to this point.

My only question is about how we could potentially quarantine the user_profile stuff? Or whether we want to keep it there for now as scaffolding.

@Archer-Key

Copy link
Copy Markdown
Collaborator Author

As for the user_profile stuff I think we could either just remove it as it's a separate app, or we could leave it for our own use such as if we want to allow the user to have a profile.

I don't think its of enough importance to require a decision to be made at the current moment.

@Precipicee

Copy link
Copy Markdown
Collaborator

How did you test your code? I'm running my own version now, but just wanted to make sure mine does everything yours does.

@Archer-Key

Copy link
Copy Markdown
Collaborator Author

So far I've just used the admin endpoint to create items and then the api endpoints (api/planner/[task/topic/time]/) to view the data and ensure its what's expected.

@Precipicee

Copy link
Copy Markdown
Collaborator

Ok, I haven't tested with data but I've tested the paths and it seemed to work. I'll accept the PR now.

@Precipicee
Precipicee merged commit 9bb2e90 into use_uv Dec 11, 2025
5 of 6 checks passed
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.

2 participants