-
Notifications
You must be signed in to change notification settings - Fork 10
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
implementation of the REST API for the event feature #14
Comments
i created this issue so that we can discuss how we plan on implementing this feature , also to decide the kind of data we will be collecting |
@wolejnr i need you here |
great. let me add the other assignees at the side. |
take a look at https://djangogirls.org/events/ to get an idea of what you would be working on. |
That i have done |
we just need to finalize on the set of data we will been rendering |
I am with you @adekoder.
I think we can split location into |
"gatsbyJS" i have no idea what it is but will do my research, |
i thought we are only implementing the API why do we need this "gatsbyJS" |
yeah its the Api you would be concerned about. That would be done in django |
then why do we need "gatsbyJS" |
You are working on APIs not the frontend. It is the frontend team that needs to learn gatsbyjs. Not all part of the site would be built with django. In fact django would just be exposed as a rest Api only. We are not using the templates. Gatsby is a static generator site that ensures we are just deploying HTML and CSS and JavaScript in d end, no server to host. The whole django setup is useful from an admin point of view not from a user/frontend view. |
Thanks for the clarification @gbozee |
@adekoder based on your comment:
Just a thought especially since we are trying to pattern after DjangoGirls. |
ok, we can add it if we want, but i don't see any reason to |
So my suggestions as regards the data to be rendered for events are given as follows:
Check those that you agree with and include those you think should also be there. |
@adekoder Do you agree with the suggestions for data to be rendered for events?? |
sure |
Based on the list you gave @wolejnr There are some fields i am not sure would be required yet. Try reaching out to @shante66 for more information on what to consider here |
No description provided.
The text was updated successfully, but these errors were encountered: