-
Notifications
You must be signed in to change notification settings - Fork 138
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
Dictionary Site #21
Comments
Hi @ijemmao I'd love to work on this issue. Anything I should know before getting started? |
Amazing! Yeah, there's a few things that I was considering for this ticket:
These bullets are super high-level and there's definitely a ton of details that need to be worked out, so please let me know if you have any questions. I recommend creating smaller tickets that focus on a particular part of building out the front-site so others can help out. |
Ah. I've never actually worked with Gatsby before, so if that's a requirement I'm not sure I'll be able to pitch in. |
@C-Zekeri A lot of this is still up the air, so if there's a framework that you prefer using you can suggest it. But if you're not feeling up to it no worries! You can still put in feature requests for the API! |
Alright then. On second thought, since I have a bit of React knowledge, I'd be willing to learn Gatsby for the project. The only thing is I might take a bit more time, but if you don't mind that, then I'll be glad to do so. |
@C-Zekeri nice! In order to make this more manageable, I recommend making super small issues that break up the work that your going to be completing. So a good first issue would be to create a new branch that's dedicated to the source code to hold the site and set up a basic Gatsby project there. |
Alright, will get on it, thank you very much. |
Hi @ijemmao. Are we also including an about page, or the lone home page with search results linking to a detail page? |
You can include an about page but we'll have to discuss on your PR what makes sense to be on the about page. I think what would be better if you separated the about page out to its own PR. I also set up a really basic Gatsby site on my local branch, I can push my code and you could start iterating from it. You should see a |
Great. Thank you! |
Hello I will be working on enhancing the frontend, my first step will make the search button to look for welcoming. |
@Mancancode awesome! Could you create an issue that details what you're going to do exactly, and I can officially assign you to that ticket? |
Closing this ticket since it's outdated and a good portion of what's been mentioned in this thread has been completed. |
Create a quick site that uses the API and displays the results.
The text was updated successfully, but these errors were encountered: