Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Django tweaks #40

Closed
mjaquiery opened this issue Feb 22, 2023 · 0 comments · Fixed by #60
Closed

Django tweaks #40

mjaquiery opened this issue Feb 22, 2023 · 0 comments · Fixed by #60
Assignees
@mjaquiery
Labels
backend Python Django/DRF backend v2.0.0 Required for v2.0.0
Milestone
v2.0.0

Comments

@mjaquiery
Copy link
Collaborator

  • Entity creation logic should be handled by Django Serializers rather than Views
  • Harvester ENV stuff is no longer handled, may be useful to be able to specify a base path.
    • Could be done in config/Harvester view
  • Might be nice to have an Updated Entity Relationship Diagram
@mjaquiery mjaquiery added the v2.0.0 Required for v2.0.0 label Feb 22, 2023
@mjaquiery mjaquiery added this to the v2.0.0 milestone Feb 22, 2023
@mjaquiery mjaquiery added the backend Python Django/DRF backend label Feb 22, 2023
@mjaquiery mjaquiery mentioned this issue Feb 24, 2023
Merged
mjaquiery added a commit that referenced this issue Feb 24, 2023
@mjaquiery
Backend/django (#45)
5be0842 
Major update. 

## Key features:
- Moved backend over from Flask to Django
  - Streamlines database interaction
  - Provides a more reliable framework for code
- Converted frontend to TypeScript
  - More robust code
- Separated Harvester code from backend code
  - Harvesters now rely on API endpoints rather than direct database writes
- Updated documentation throughout

## Missing features (todo):
#39 
- Frontend tests
- Cleaning out old frontend files

#40 
- Entity creation logic should be handled by Django Serializers rather than Views
- Harvester ENV stuff no longer handled 
  - Mostly matters for being able to specify a base path; could be done in config/Harvester view
- Updated Entity Relationship Diagram

#38 
- Python API client code
  - Should be automatically generated via swagger codegen API/docker container, perhaps on demand?

#41 
- Migration from old database

#42 
- API token management (including long-term tokens for scripts)
- User management (reset password, update email?)

#43 
- Profiling important things
  - Data upload
  - Data download
@mjaquiery mjaquiery mentioned this issue Feb 24, 2023
Merged
@mjaquiery mjaquiery self-assigned this Mar 3, 2023
@mjaquiery mjaquiery mentioned this issue Mar 8, 2023
Merged
@mjaquiery mjaquiery linked a pull request Mar 8, 2023 that will close this issue
Django tweaks #60
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mjaquiery mjaquiery

Labels
backend Python Django/DRF backend v2.0.0 Required for v2.0.0
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging a pull request may close this issue.

Django tweaks Battery-Intelligence-Lab/galv
1 participant
@mjaquiery