search

Seneca-CDOT / ostep-dashboard

Displays daily updates for the OSTEP team.
MIT License
4 stars 9 forks source link

convert css files to sass and refactor code #61

Closed miggs125 closed 4 years ago

miggs125 commented 5 years ago

The style files are, specifically App.css is getting rather large and needs to be refactored and modulated in order to keep the codebase maintainable as the project scales

Silvyre commented 5 years ago

Hello,

I am a Seneca student currently taking Topics in Open Source Development. I have chatted with my professor (@humphd) about contributing to an OSTEP repository for Hacktoberfest, and working on this issue for the OSTEP dashboard seems like a great opportunity to do so.

If possible, may I request permission to work on this issue? I would alternatively be happy to drop by CDOT in person to chat more about how I may help out.

Thank you very much,

Rafi

humphd commented 5 years ago

cc @manekenpix and @miggs125, who are both in my open source class, and also work or have worked in this repo as well. Perhaps one of you can help guide @manekenpix so he can contribute too?

poftadeh commented 5 years ago

@Silvyre feel free to take on this issue for Hacktoberfest.

Silvyre commented 5 years ago

Thank you very much, everyone! I look forward to working with you!

manekenpix commented 5 years ago

@humphd yes, no problem. @Silvyre let us know if there's anything we can help you with.

miggs125 commented 5 years ago

@Silvyre feel free to take it, I started working on it some time ago on this branch. Some of the content on the branch may be outdated. It's been a while since I worked on it.

Silvyre commented 5 years ago

@manekenpix @miggs125 Thank you! I'll certainly be in touch!

Silvyre commented 5 years ago

Thanks to the help of @miggs125, I have been able to successfully run the OSTEP Dashboard locally via npm run-scriptdebug.

Within the next three weeks, I plan to apply the knowledge I am gaining from a Udemy course (also recommended by @miggs125) to finish the work started on the issues/61-convert-css-to-sass branch.

poftadeh commented 5 years ago

@Silvyre good to hear. I've changed #10 to an open help wanted issue as well if you're in need of more contributions toward Hacktoberfest. It's a relatively small starter issue that you or anyone else from @humphd 's class are free to take on if interested.