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

Course Announcement Screen #284

Closed sakshi-choudhary closed 3 years ago

sakshi-choudhary commented 3 years ago

ISSUE #257

Describe the changes you've made Developed Course Announcement Screen and comments page

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)

Additional context (OPTIONAL) Linked the announcement page created under more options in my courses

Checklist

Provide a Deployed link of route/page that needs to review Preview: https://deploy-preview-284--festive-allen-a7f545.netlify.app/mycourses/:coursename/announcement

Screenshot 2021-04-24 at 00 50 20 Screenshot 2021-04-24 at 00 50 39
sakshi-choudhary commented 3 years ago

@Abhishek-kumar09 Please review

Abhishek-kumar09 commented 3 years ago

It doesn't look like a course announcement page, but more like discussion forum.

sakshi-choudhary commented 3 years ago

It doesn't look like a course announcement page, but more like discussion forum.

Okay so I actually took inspiration from the google classroom. Here, in this page only the course mentor can post something, like make an announcement, share some materials, and students can comment. Students can't post anything. SO, this was the idea. If it doesn't make sense please let me know I will update this into the discussion forum page as anyways the discussion forum's UI is not good which is there right now. @Abhishek-kumar09

sakshi-choudhary commented 3 years ago

I will use this card component in the discussion forum and update this page with like a announcement notification thing.

Abhishek-kumar09 commented 3 years ago

You can try creating design on Figma first, that will save a lot of effort of writing code again and again. We can use the courseCards and show that they are upcoming, or having some announcements within it.

If the student are enrolled to a course, then there announcement may land into their profile/announcements.