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

Add shared package for database models #671

Merged
merged 39 commits into from
Apr 10, 2024
Merged

Conversation

AndyRae
Copy link
Member

@AndyRae AndyRae commented Apr 9, 2024

Changes

  • Moves the database models to a app/shared package
  • Add the shared package to the webapp and workers dependencies, updating imports accordingly
  • Add a database test function to test with a deployed Azure function

Closes #662
Tracked by #612

The idea will be to use shared for any shared code between Django / Azure functions.
Retaining the app/api/data package for now, as it's attached to migrations (a future problem)

So these changes are mostly changing imports, and the interesting bits are dependency changes.

Checks

Important: please complete these before merging.

  • Run migrations, if any.
  • Update changelog.md, including migration instructions if any.
  • Run unit tests.

Copy link

github-actions bot commented Apr 9, 2024

Tests Skipped Failures Errors Time
25 1 💤 0 ❌ 0 🔥 2.104s ⏱️

@github-actions github-actions bot removed the Area: CI label Apr 9, 2024
@github-actions github-actions bot removed the Area: CI label Apr 10, 2024
@AndyRae AndyRae merged commit b95ee2a into master Apr 10, 2024
12 checks passed
@AndyRae AndyRae deleted the feat/662/data-package branch April 10, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Backend Area: Workers dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Django ORM use with Azure Functions
3 participants