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.

Sign up for GitHub

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

Jump to bottom

Add new resources to learn machine learning in the React app. #1

Open
Parasgupta44 opened this issue Sep 29, 2020 · 8 comments
Open

Add new resources to learn machine learning in the React app. #1

Parasgupta44 opened this issue Sep 29, 2020 · 8 comments
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Contribute!!

Comments

@Parasgupta44
Copy link
Owner

Task

Add new resources to learn machine learning from. Please check Readme.md and Contributing.md in the repo's root for more details.
@Parasgupta44 Parasgupta44 added enhancement New feature or request good first issue Good for newcomers hacktoberfest Contribute!! labels Sep 29, 2020
@Parasgupta44 Parasgupta44 changed the title Add new resources for learning machine learning in the React app. Add new resources to learn machine learning in the React app. Sep 29, 2020
@Parasgupta44 Parasgupta44 reopened this Sep 30, 2020
@CiTRuS93 CiTRuS93 mentioned this issue Sep 30, 2020
Merged
@sloorush sloorush mentioned this issue Sep 30, 2020
Merged
@supanthapaul
Copy link
Contributor

supanthapaul commented Sep 30, 2020 •
edited
Loading

Added new resources from mathematicalmonk and David Silver. Please review the PR.
#12

@Parasgupta44
Copy link
Owner Author

Parasgupta44 commented Sep 30, 2020 •
edited
Loading

Added new resources from mathematicalmonk and David Silver. Please review the PR.
#12

Merged for deployment to master !
Thanks.

Parasgupta44 pushed a commit that referenced this issue Oct 1, 2020
@CiTRuS93
Merge pull request #1 from Parasgupta44/master
4fa0533 
sync
Parasgupta44 pushed a commit that referenced this issue Oct 1, 2020
@Jawkx
Merge pull request #1 from Parasgupta44/master
e66fa96 
Update from main
@CiTRuS93
Copy link
Contributor

CiTRuS93 commented Oct 2, 2020

Maybe we can automate this with some kind of form?

@Parasgupta44
Copy link
Owner Author

Yeah...that's a nice idea.
So, do you have any ideas how should we go for it because my plan was to keep the resources content as static as possible (that's why no db), so that reviewing them for sanity would be much easier !

@CiTRuS93
Copy link
Contributor

CiTRuS93 commented Oct 2, 2020

Maybe writing into some "suggestions.json" and review before adding to resources.json?

@Parasgupta44
Copy link
Owner Author

Maybe writing into some "suggestions.json" and review before adding to resources.json?

Yeah, that seems like a good option.

I guess db setup n stuff can be an overkill for this (?)
Or maybe not much of a hassle ?

@CiTRuS93
Copy link
Contributor

CiTRuS93 commented Oct 4, 2020

I don't think we really need a db cause it's only one table.
The bigger issue is how to prevent spam in this form.

@archonn archonn mentioned this issue Oct 4, 2020
Merged
@Parasgupta44
Copy link
Owner Author

The bigger issue is how to prevent spam in this form.

Yeah, this one is crucial. Will look into this for sure !!

Parasgupta44 pushed a commit that referenced this issue Oct 18, 2020
@taichagas
Merge pull request #1 from Parasgupta44/master
6404f45 
Merge original with forked
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Contribute!!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@CiTRuS93 @supanthapaul @Parasgupta44