-
Notifications
You must be signed in to change notification settings - Fork 156
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
Scholarships page for the website #56
Comments
Hi @umang-sinha what a great initiative! Can I work on this? |
Hey @syauqy thanks for deciding to contribute. I am assigning this to you. You may go ahead and start working on it. |
Hi @syauqy I'm assuming you'll be making the figma design(or any other software) and CSS/react code both yourself? |
Hi @shikha-16 since the project only has the wireframe. I think it's better if it push to the high fidelity first. I can help you with Figma for this page as long as we have decided the design guidelines to make sure it doesn't have different style compare to other pages. And also for the react/Gatsby I think I need to wait for the main project to setup first then I can working on this page. What do you think? |
Hey @syauqy We have some people working on the website designs. In this issue you are only expected to create a basic skeleton of the page. Further enhancements will be done only once the final designs for the other pages are ready. However you can take a look at the carousel design for the homepage to get a rough idea : https://github.com/shikha-16/Women-in-Technology/tree/master/documents/design/carousels |
Hi @shikha-16 and @umang-sinha please check this video of my progress on this page https://www.loom.com/share/f5a2a981d043428387c0fd68e31425a9 |
Hi @syauqy WOW that looks great. thank you for this amazing work. We also have the carousel designs ready: https://github.com/shikha-16/Women-in-Technology/tree/master/documents/design/carousels if you want to use those (it's fine if you don't rn) Everything else looks great to me! Add the logo and favicon, let @umang-sinha also comment, and then you're good to go! |
Hey @syauqy This looks perfect to me. This is exactly what we wanted. Great job! As @shikha-16 mentioned, please add the logo and favicons and then you can proceed to make a pull request. |
please check my PR here #63 . Thanks @umang-sinha @shikha-16 |
@syauqy hi, please check out the comment on your PR |
@syauqy maybe you would like to take up another page on the website? we really liked your work, and hope you can contribute more. |
Hi @shikha-16 thank you for the offer. Sure, but I think I cannot do it fast. Since I'm only able to work on it on the weekend. Let's see |
Contributors interested in working on the scholarships page can take up this issue. You can have a look at the wireframes here.
The scholarships page needs to have the following components:
Contributors are requested to strictly adhere to the format followed in the wireframes. Please go through Contributing.md and the basic skeleton before contributing.
Also join the Discord server so that communication can be done smoothly.
For an overview of the project, see here
The text was updated successfully, but these errors were encountered: