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

Merge develop into main #98

Merged
merged 50 commits into from
Aug 24, 2024
Merged

Merge develop into main #98

merged 50 commits into from
Aug 24, 2024

Conversation

tomaspalma
Copy link
Member

No description provided.

diogotvf7 and others added 30 commits February 21, 2024 12:06
Changed the name of the professor model values
Endpoint to return course unit based on id
Fixes wrong command on README and more detail on django caveats
Made redis url be constructed from environment variables and hostname not bounded to repository folder's name
Use more django orm features to speed up classes retrieval
@tomaspalma tomaspalma requested a review from a team August 20, 2024 21:03
Process-ing
Process-ing previously approved these changes Aug 23, 2024
Copy link
Contributor

@Process-ing Process-ing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀 👍 2️⃣ 🤳

thePeras
thePeras previously approved these changes Aug 23, 2024
Fix class/ endpoint to return all professors
@tomaspalma tomaspalma dismissed stale reviews from thePeras and Process-ing via 73c906f August 23, 2024 23:44
@tomaspalma tomaspalma merged commit 2e85bb4 into main Aug 24, 2024
1 check 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.

6 participants