-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve routing #342
Open
maltehuebner
wants to merge
18
commits into
main
Choose a base branch
from
improve-routing
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Improve routing #342
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* main: (52 commits) Add loggly. And they’re attributes now. This is called EnumType now. Update maltehuebner/impress-bundle Update recipe for symfony/routing Fix symfony versions to ^6.2 Fix param converter attributes. Remove sensio/framework-extra-bundle Cleanup dependencies. Deprecate getDoctrine() call. Fix missing getDoctrine() call. Fix main request. Fix all the parameter annotations. Update openapi imports. Remove composer/package-versions-deprecated. Remove fahrverbote from menu. Remove fahrverbote from city page. Remove fahrverbote from City entity. Add migration. Remove a big bunch of security. ... # Conflicts: # config/routing/6_city.xml # src/Controller/CityController.php # src/Controller/StationController.php
* main: (22 commits) Fix request bodies. Fix config. Add typed properties for Station entity. Add typed properties for Network entity. Add typed properties for Data entity. Add typed properties for City entity. Cleanup. Use AsCommand attribute. Raise elasticsearch to 7.17.2 Fix cards. Fix broken shadow url. Add more margin. Fix broken action for menu search impiut. Fix frontpage input group. Fix github banner. Fix breadcrumbs. Fix webpack and handlebars issues. Fix menu. Fix menu, remove knp menu bundle. Tmp fix for getting yarn done. ...
* main: (29 commits) Fix extramarkers url. Hide city list for today. Do not fetch stations from luftdaten. Re-enable old data cache. This adds the pin again, which is needed for elasticsearch station lookup. Improve station handling. Also handle station data in display. This brings a big speed bump. Reduce complexity. Yay, fix broken flush in wrong line. Also load different providers. Improve query. Fix put controller. Check station data before persisting. Also pass latitude and longitude to frontend. Fix broken declare() Fix config parameters. Introduce ChainedPersister. Technischer Durchstich. Fix more relations. ... # Conflicts: # src/Controller/Api/StationApiController.php
* remove-jms-serializer: (140 commits) These must be nullable. Skip null values during deserialization as well. Reorder routing. Forgot to fix implementation. Use constant for 'json' format parameter. Use constant instead of 200 http code. Move ManagerRegistry and LuftSerializer to class members. Do not expose null values. Update dependencies. Fix declarations. Introduce LuftSerializer again. Small cleanup. Fix card display. Remove UniqueStrategy. Remove Model for now. Remove DataList. Remove PollutionDataFactoryStrategy stuff as it is covered by postgis. Simplify data retriever stuff. Rename even more code. Fix more services. ... # Conflicts: # config/routing/2_api.xml # src/Controller/AnalysisController.php # src/Controller/Api/AnalysisApiController.php # src/Controller/Api/CityApiController.php # src/Controller/Api/DisplayApiController.php # src/Controller/Api/StationApiController.php # src/Controller/Api/ValueApiController.php
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.