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

#10 - courses crud #55

Merged
merged 13 commits into from
Oct 4, 2023
Merged

#10 - courses crud #55

merged 13 commits into from
Oct 4, 2023

Conversation

kamilpiech97
Copy link
Member

@kamilpiech97 kamilpiech97 commented Oct 3, 2023

Should close #10.

Added CRUD for courses.
Added tests.
Fixed toggling activity on semesters.

Index view:
image

Create view:
image

Edit view:
image

@kamilpiech97 kamilpiech97 marked this pull request as ready for review October 3, 2023 13:48
@kamilpiech97 kamilpiech97 requested a review from a team as a code owner October 3, 2023 13:48
app/Enums/ClassType.php Outdated Show resolved Hide resolved
app/Enums/StudyForm.php Show resolved Hide resolved
app/Http/Controllers/Dashboard/CourseController.php Outdated Show resolved Hide resolved
database/factories/CourseFactory.php Outdated Show resolved Hide resolved
kamilpiech97 and others added 2 commits October 3, 2023 19:30
Co-authored-by: Krzysztof Rewak <krzysztof.rewak@blumilk.pl>
app/Enums/ClassType.php Outdated Show resolved Hide resolved
app/Enums/Semester.php Outdated Show resolved Hide resolved
app/Enums/ClassType.php Outdated Show resolved Hide resolved
# Conflicts:
#	resources/js/Shared/Forms/Select.vue
#	routes/web.php
@kamilpiech97 kamilpiech97 merged commit 65a28f9 into main Oct 4, 2023
3 checks passed
@kamilpiech97 kamilpiech97 deleted the #10-courses-crud branch October 4, 2023 09:34
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.

Courses management
3 participants