Skip to content

Courses Details Endpoint (#40) #81

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Courses Details Endpoint (#40) #81

wants to merge 1 commit into from

Conversation

blu3eee
Copy link

@blu3eee blu3eee commented Apr 9, 2024

  • Added endpoints for course searching and get course info
  • Json files are stored in a separated repo and when a server starts, it fetches the courses from the repo, checks the current Curriculum table, and inserts the courses. The logic to update courses on server start can be further implemented [SECONDARY FEATURE] Advanced logic to trigger courses update process #80

@blu3eee
Courses Details Endpoint (#40)
0b4115f 
this commit should resolve #40
@blu3eee blu3eee force-pushed the leaf/blu3eee/courses-details branch from a55965e to 0b4115f Compare April 9, 2024 00:51
@blu3eee blu3eee mentioned this pull request Apr 9, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PattyCakesxD PattyCakesxD Awaiting requested review from PattyCakesxD

@Leaversa Leaversa Awaiting requested review from Leaversa

@GuyWhoCode GuyWhoCode Awaiting requested review from GuyWhoCode

@JRSiwiecki JRSiwiecki Awaiting requested review from JRSiwiecki

@OJ101003 OJ101003 Awaiting requested review from OJ101003

@alimomennasab alimomennasab Awaiting requested review from alimomennasab

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@blu3eee