8000 feat: Course Pathway by rotimi-best · Pull Request #528 · classroomio/classroomio · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Course Pathway #528

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

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

feat: Course Pathway #528

wants to merge 46 commits into from

Conversation

rotimi-best
Copy link
Collaborator

What does this PR do?

Fixes # (place issue number here without bracket)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change adds a new database migration
  • This change requires a documentation update

How should this be tested?

  • Test A
  • Test B

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand bits
  • Ran pnpm build
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the ClassroomIO Docs if changes were necessary

rotimi-best and others added 19 commits July 16, 2024 17:12
* feat: Pathway Pages UI

* fix: remove console logs

* fix: PR Reviews

* Update +layout.svelte

* Update index.ts

* PR Fixes

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* worked on the pathway sidebar and the empty state

* worked on the card and modal

* worked on the sidebar path

* dashboard UI and translations all done

* updated the translations

* feat: show submissions for live classes

* updated the name to pathways and the translations

* updated the name to pathways and the translations

* fix: lesson widget alignment on page (#396)

* fix

* fix

* fixed overlapping issue

* removed console logs

* removed extra line

* chore: modifies seed data to verify email (#395)

* feat: RLS on all tables (#399)

* implement RLS

* set default course currency to USD

* remove unused import

* update default currency in course in db

* worked on the reviews

* updated the prop for the pathway card

* fix: auth issue due to org insert rls policy

* fix: add style for native select css (#403)

* fix: validation on profile page (#398)

* fix validation of username and fullname

* fix

* fix message

* Update validator.js

* Update Profile.svelte

* remove unused functions

---------

Co-authored-by: Rotimi Best <best@classroomio.com>

* chore: update translations

* fix: #404 autosave issue on lesson

* docs: start supabase with pnpm

* feat: add 4 certificate templates (#295)

* add cloud translation api package

* feat: Script that auto generates translations for other languages

* add comments

* fix: broken translation

* improve translation

* update locck file

* clean up

* all round trasnlations check

* checkpoint

* finish up

* move to dev dependencies

* fix vercel build error

* translations fix

* - add documentation
- translation bug fix

* quick adjust

* PR Fixes

* add new line

* update docs

* checkpoint

* feat: redesign & addition of certificates

* text color update

* patterned width fix

* "student-name" class remove

* checkpoint

* PR Fix

* quick fix

* PR Fixes

* update certificate ready page for students

* PR Fixes

* quick cleanup

* move fetchCourses into .ts file

* feat: give students access to certificates only after completing course

* checkpoint

* height fix

* Merge branch 'main' of https://github.com/rotimi-best/classroomio into classroomio-issue-272

* Fix; merge PR Fix

* fix: width of plain & professional certs

* PR Fix

* feat: add translation

* handle error when trying to download a certificate

* add fetchExploreCourses

* fix: requested reviews

* review fixes checkpoint

* replace the certificate background

* height fix for patterned certififcates

* fix more bugs

* rename schema

---------

Co-authored-by: Rotimi Best <best@classroomio.com>

* feat: teacher email should include student details

* Update based on reviews

* Update based on reviews

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
Co-authored-by: Smriti Doneria <smritidoneria@gmail.com>
Co-authored-by: Vanessa Osuka <77996535+cannarron@users.noreply.github.com>
Co-authored-by: Dmytro <31030855+Laurent-di@users.noreply.github.com>
Co-authored-by: Adeyemi Adetunji <tvnji01@gmail.com>
* worked on the pathway sidebar and the empty state

* worked on the card and modal

* worked on the sidebar path

* dashboard UI and translations all done

* updated the translations

* updated the name to pathways and the translations

* updated the name to pathways and the translations

* done with the create Pathway

* donw with the create pathway

* wrapped up the api Integration for the dashboard

* worked on the pathway card

* added translations to the pathway modal

* reviewed mmy code and made some updates

* deleted svg file

* more review chnages

* more reviews

* More Reviews

* added the migration file

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* fix:
- Change pathways route
- Fix  Navigation Header glitch
- Replace pathway sidebar with existing component
- Replace pathways course card with existing component
- Fix NavExpandable icon bug

* - getPathwayNavItemRoute type fix
- add translations

* include "add" prop

* fix: navbar styling

* Chang courseCompletion to CourseCompletion

* Change courseCompletion to CourseCompletion

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* checkpoint

* feat: Pathway Certificate Page
* feat: Add Course Modal

* feat: add translation

* fix: feedback fix

* PR Fixes

* feat: Merge Fix

* fix: merge fixes

* feat: Arrange Course Modal (#413)

* fix: merge bug fixes

* feat: Add course modal

* chore: refactor modal and textfield to be much simpler

* remove unused var

* refactor

* remove pathways folder

* rename from PathWay to Pathway

* delete wrong casing files

* rename files

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* worked on the pathway sidebar and the empty state

* worked on the card and modal

* worked on the sidebar path

* dashboard UI and translations all done

* updated the translations

* updated the name to pathways and the translations

* updated the name to pathways and the translations

* done with the create Pathway

* donw with the create pathway

* wrapped up the api Integration for the dashboard

* worked on the pathway card

* added translations to the pathway modal

* working on the Ui for pathway in LMS

* worked on the UI of the LMS pathway

* delete unused stuff

* more merge fix

* restore org/pathway

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* feat: Merge Fix

* PR Fixes

* translations fixes
* checkpoint

* feat: add RLS for certificate ui

* translations

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* finished with the backend of the LMS pathway

* refactor

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* feat: People Page

* refactor

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* working on the landing page

* feat: pathways landing page

* feat: translations

* quick fixes

* undo de

* undo translations

* improve types

---------

Co-authored-by: emmanuel <chifez1@gmail.com>
Co-authored-by: Rotimi Best <best@classroomio.com>
* feat: Pathways News Feed

* refactor

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* finished the settings page

* round up settings page

* update translation

* translation

---------

Co-authored-by: Rotimi Best <best@classroomio.com>
* feat: Pathways News Feed

* feat: completion of newsfeed integration

* feat: courses page

* merge fixes

* checkpoint: people's page completion

* feat: course landing page integration

* working on the LMS side

* feat: bug

* code clean up

* Lms pathway fully done

* fix; navigation id for courses

* fix: unlocking courses bug

* wrapping things up

* still cleaning things up

* merge fix

* feat: certificate display

* feat: db migration

* update seed

* Fixed the setting update issue

* finished the unlocking of pathway

* checkpoint: certificates update

* feat: bug fixes

* feat: finished the explore courses of the LMS page (#454)

* worked on the explore courses

* worked on the explore courses

* Finished the explore page

* updated the migration file

* Removed the comment and unneeded console.log in api.ts

---------

Co-authored-by: emmanuel <chifez1@gmail.com>
Co-authored-by: Rotimi Best <best@classroomio.com>
Copy link
vercel bot commented Dec 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cio-com ❌ Failed (Inspect) May 19, 2025 7:25am
cio-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2025 7:25am

Copy link
github-actions bot commented Dec 26, 2024

Thank you for following the naming conventions for pull request titles! 🙏

Copy link
cloudflare-workers-and-pages bot commented Jan 2, 2025

Deploying courseapp with  Cloudflare Pages  Cloudflare Pages

Latest commit: 58e4191
Status:🚫  Build failed.

View logs

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.

3 participants
0