Skip to content
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

[FEATURE] : Removing manually updated contributors to dynamically updated contributors #98

Open
PritamSarbajna opened this issue Jun 6, 2023 · 6 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@PritamSarbajna
Copy link
Owner

Is your feature request related to a problem? Please describe.

The contributor section needs to be manually updated whenever any new contributor contributes to it. So it becomes problem and there is a issue always open for this.

Describe the solution you'd like

Okay so we can remove the manual entry section and make use of https://github.com/PritamSarbajna/tourism-website/graphs/contributors, which can tell us about the contributors to the project.

Additional context

We have to change the UI accordingly and check if the UI really suits for the website.

@PritamSarbajna PritamSarbajna added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jun 6, 2023
@amolgorithm
Copy link
Contributor

@PritamSarbajna Yeah, this can be done relatively simply but the project would not be Pure JS. You would need Node.js to use REST API to request a fetch for Github repo data (e.g. contributors). Using Node.js will require some major adjustments to the code though, such as using an engine like EJS for a responsive front-end html page that is linked to the back-end server. In which case, you would need Node.js, node modules like Express and EJS. You would also need to replace the index.html page with an EJS file. So ultimately, the whole project needs to be converted to a Node.js project, and you require the REST API. This will require a lot of adjustments and changes, but it should be able to do, theoretically.

@PritamSarbajna
Copy link
Owner Author

@amolgorithm What you're saying is correct but for that we have to change the whole project to a node js project. But I want to make this project beginner friendly because there are a lot of folks who are contributing is relatively new in this field. So instead if you want to really fix it, I can make a API with FastApi and host it anywhere. I will provide you the endpoints so you can easily fetch the data. Is it okay with you ?...In that way we have to just make a fetch function in javascript and change a few lines of code in this project.

@amolgorithm
Copy link
Contributor

@PritamSarbajna That sounds like a good idea! That is indeed okay with me. Github recommended REST, albeit Fast API should work more efficiently. That should work, we could try it out.

@PritamSarbajna
Copy link
Owner Author

Okay @amolgorithm, My sem exams are going on that's why I'm not active so much but give me few days once the exam finishes I will make the API and get back to you.

@amolgorithm
Copy link
Contributor

Hey @PritamSarbajna, now that the learn more issue is done, we can move onto this. Let me know when you make the API and I can begin working on this issue.

@PritamSarbajna
Copy link
Owner Author

Hey @PritamSarbajna, now that the learn more issue is done, we can move onto this. Let me know when you make the API and I can begin working on this issue.

Yeah I will let you know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants