Added: Student Dashboard Screen

codeforcauseorg / edu-client

Product focuses on 100% education as well as upskilling developing countries and rural areas.

https://educlient.codeforcause.org/

MIT License

130 stars 149 forks source link

Added: Student Dashboard Screen #214

Closed adarsh-technocrat closed 3 years ago

adarsh-technocrat commented 3 years ago

Issue Number

fixes #135

Describe the changes you've made

Created Student Dashboard Screen and some of the components are left to add like

[x] Assignment Submission Section
[x] Job page

I'll create a separate PR for the respective sections that will resolve this issue!

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

Additional context (OPTIONAL)

Test plan (OPTIONAL)

A good test plan should give instructions that someone else can easily follow.

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

adarsh-technocrat commented 3 years ago

@Abhishek-kumar09 ping!

adarsh-technocrat commented 3 years ago

Follow camelcasing while naming variables or classes etc.

Thanks, @Abhishek-kumar09 I'll keep in mind :)

adarsh-technocrat commented 3 years ago

Sorry, @Abhishek-kumar09 hitted accidentally close with the comment option!

adarsh-technocrat commented 3 years ago

@Abhishek-kumar09 ping!

Abhishek-kumar09 commented 3 years ago

There are merge conflicts. You can resolve these in browser itself. This extension also helps: https://addons.mozilla.org/en-US/firefox/addon/refined-github-/

adarsh-technocrat commented 3 years ago

There are merge conflicts. You can resolve these in browser itself. This extension also helps: https://addons.mozilla.org/en-US/firefox/addon/refined-github-/

Thanks @Abhishek-kumar09 I'll do the changes :)

Abhishek-kumar09 commented 3 years ago

Also, create a new PR to update PR template which asks for giving links to the deploy preview of the route/page to be reviewed

adarsh-technocrat commented 3 years ago

Also, create a new PR to update PR template which asks for giving links to the deploy preview of the route/page to be reviewed

Sure @Abhishek-kumar09 I'll create PR for that :)

adarsh-technocrat commented 3 years ago

https://user-images.githubusercontent.com/47661086/114208694-63d71800-997b-11eb-9819-e852c0dc942b.mp4

Hey, @Abhishek-kumar09 I think now this PR is good to merge!

Abhishek-kumar09 commented 3 years ago

Good Work!