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

Remove concept of liveCourseAccess from Course Introduction COmponent #1177

Open
3 tasks
eleanorreem opened this issue Oct 31, 2024 · 4 comments
Open
3 tasks
Assignees
Labels
complexity: beginner Time needed to do this ticket is less than 1 day. good-first-issue Good for newcomers javascript Pull requests that update Javascript code maintenance Maintenance / chore work state: approved Ready to go. Not blocked or pending.

Comments

@eleanorreem
Copy link
Contributor

eleanorreem commented Oct 31, 2024

Overview

We used to have a concept of a "Live course". This was a course that was being delivered live rather than being available online for everyone at any time. This concept has been retired and we want to remove it from the code to simply. This ticket is to remove the concept of liveCourse from the CourseIntroduction component.

Action Items

  • Remove liveCourseAccess concept from the CourseIntroduction component. This includes courseLiveNow or courseLiveSoon booleans.
  • Check everything looks right on course page where the component is rendered
  • Run cypress tests to ensure that everything is working.
@eleanorreem eleanorreem changed the title Remove concept of liveCourseAccess from Remove concept of liveCourseAccess from Course Introduction COmponent Oct 31, 2024
@eleanorreem eleanorreem added good-first-issue Good for newcomers complexity: beginner Time needed to do this ticket is less than 1 day. maintenance Maintenance / chore work state: approved Ready to go. Not blocked or pending. javascript Pull requests that update Javascript code labels Oct 31, 2024
@druxvh
Copy link

druxvh commented Nov 23, 2024

/assign it to me :)

Copy link
Contributor

Thank you @druxvh you have been assigned this issue!
Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request. ✨


Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you!
Learn more about Chayn's impact here. 🌸

@druxvh
Copy link

druxvh commented Nov 27, 2024

While creating partner admin account for the local bloom users env variables on the admin/dashboard route I can't select any partner nor I'm able to create an admin account.

Image

Image

@eleanorreem
Copy link
Contributor Author

@druxvh thank you for all your efforts so far. Have you seen the contributor wiki? https://www.notion.so/chayn/Chayn-Tech-Contributor-Wiki-5356c7118c134863a2e092e9df6cbc34?pvs=4#eb2e24de94bd451f8683abe496656013. There are instructions on how to seed the database.

let me know if you have any more questions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: beginner Time needed to do this ticket is less than 1 day. good-first-issue Good for newcomers javascript Pull requests that update Javascript code maintenance Maintenance / chore work state: approved Ready to go. Not blocked or pending.
Projects
None yet
Development

No branches or pull requests

3 participants