search

codeforcauseorg / edu-client

Product focuses on 100% education as well as upskilling developing countries and rural areas.
https://educlient.codeforcause.org/
MIT License
128 stars 148 forks source link

Develop About Course Page #298

Closed sakshi-choudhary closed 3 years ago

sakshi-choudhary commented 3 years ago

Describe the solution you'd like

Develop an about course page.

Additional context

The page will be linked to this option present currently in each course's more options.

sakshi-choudhary commented 3 years ago

@Abhishek-kumar09 @vasudev19126 Can I work on this?

vasudevsinghal commented 3 years ago

@sakshi-choudhary yes

Abhishek-kumar09 commented 3 years ago

@sakshi-choudhary I think this is already created, https://edu-client.netlify.app/course/2 Are you thinking something else?

sakshi-choudhary commented 3 years ago

@sakshi-choudhary I think this is already created, https://edu-client.netlify.app/course/2 Are you thinking something else?

Okay, yeah but this for new users right? as there's an option to buy as well. So, do we need an about course page for enrolled students? as I saw this about course option the more options tab.

adarsh-technocrat commented 3 years ago

Hey, @sakshi-choudhary I don't think it will be a good idea for making a difference about course page route for this section rather what we can do is we can make that list to expanded and collapse list and show a summary of about course. This will also help users to see course details while attending the lectures.

Please let me know your thoughts on this :)

sakshi-choudhary commented 3 years ago

Hey, @sakshi-choudhary I don't think it will be a good idea for making a difference about course page route for this section rather what we can do is we can make that list to expanded and collapse list and show a summary of about course. This will also help users to see course details while attending the lectures.

Please let me know your thoughts on this :)

Yes, a collapse will be fine.

Abhijay007 commented 3 years ago

@sakshi-choudhary this issue is resolved by your PR (#309), you can close it now : )