-
Notifications
You must be signed in to change notification settings - Fork 2
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/fake-students-endpoint #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a feature to just test if we can connect our repo with our FE partners one, I think is fair enough. IDK if @jordimorillo would like to test this, but ATM I Think is ok.
Good and really fast job! 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work, I have tested the test route on local machine and everything is fine. 😸
- I have a suggestion to add to your contribution, instead of throwing the array of students directly in the routes, you can create a json file in the public folder and store the data in that file.
Example:
return response()->file(public_path('fakeData.json'));
I think it adds more elegibity and if you decide to add more fields or more students, the path file will be very loaded with data that will only be test data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing to add! Súper Rosa!
Now the students.json file is located in database/data |
Add fake route and images