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
Vehicle GET request #200
Vehicle GET request #200
Conversation
…ncy variables. Information is logged. One can not do more than one post request in one minute.
…into feature-postfeedback
Applied fixes from StyleCI
Enhancing your pull request
…ny Requests" will be returned. If the request is succesful however a "201 created" will be returned and the user will be redirected to the GET vehicle he posted to.
…into feature/postfeedback # Conflicts: # api/.htaccess # api/APIPost.php
…mbs and survey data. The README.md file changed for this reason.
|
||
use MongoDB\Collection; | ||
|
||
class SpitsgidsController |
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.
A Controller in a webapp is something that takes care of a HTTP call after that it has been processed by a router... This is not a controller, no?
It's a bit annoying that this PR also includes the other one. Let's first try to get the other PR to development and let's then look at this one! |
…elopment # Conflicts: # api/data/NMBS/vehicleinformation.php
Can you reopen this pull request without merge conflicts? |
And also make sure that if no mongodb connection could be opened, that the occupancy is not shown and the mongodb lookup is skipped. maybe we can do this using an extra .en parameter? |
Implemented following issue: #182.