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 models for sessions and tracks #9126

Open
wants to merge 3 commits into
base: dev-v3
Choose a base branch
from

Conversation

Shivangi-ch
Copy link

Fixes #9109

Short description of what this resolves:

Added models for session , tracks and other related models

Changes proposed in this pull request:

  • Added models for Tracks , Microlocations, Session, and SessionTypes inside events app.
  • Added models for VideoStreams inside video_channels app.

PS - This PR builds on top of work done in PR #9125. Please merge it before merging this PR.

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream development branch.
  • The unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • All the functions created/modified in this PR contain relevant docstrings.

@Shivangi-ch
Copy link
Author

Please review this PR!

@fillingtothemomo
Copy link

@Shivangi-ch this PR has commits of your previous PR as well so you could either squash the commits to not mess up the commit history or use different branches for both PRs I feel.

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.

None yet

2 participants