Conversation
Owner
cr3a7ure
commented
Nov 5, 2018
| Q | A |
|---|---|
| Bug fix? | yes/no |
| New feature? | yes/no |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | api-platform#1234, api-platform#5678 |
| License | MIT |
| Doc PR | api-platform/docs#1234 |
ApiPlatformExtension cleanup
Remove a now useless composer hack
Add tests on doctrine collection purge
Force Yaml::dump to dump empty array as actual empty array
…eys-in-normalization-context [EagerLoadingExtension] Remove unset attributes key in normalization context
Unit test also added
* Bump PHPStan analysis to level 6 * Bump PHPStan analysis to level 6
Bump PHPStan analysis to level 6
Improve graphiql CSS
…refix Prefix root resource's route_prefix to sub-resources
) * Allow user-defined cache headers A user may wish to define response cache headers in a custom controller or per resource. E.g. a /entity/random endpoint should have a max-age of 0 * Override cache max-age and shared-max-age Add annotation attributes for cache_headers which allows max_age and shared_max_age to be assigned per resource. * Remove unused property The property was added when I thought it may be good to have the option outside of 'attributes' - decided to put the option in attributes instead and failed to remove this property. * Code style fixes * Test code style fix * Review changes applied - Order of constructor arguments + default null value - Remove 'throws' annotation - setting $resourceCacheHeaders with default fallback to empty array - Improved conditional statements * Add cacheHeaders Attribute annotation * Accidental typo * Alphabetical order Updated attribute annotation to alphabetical order - moved the property into alphabetical order as well
Deprecate dead code in QueryJoinParser and remove internal usage
…ilter-over-multiple-values allow to use multiple values with NumericFilter
Allow multiple order filters in entity.
fix OneToOne bidirectional + unitTest WIP
Allow an input and an output for a given resource class
[Bugfix] Remove type Error in the Debug Toolbar
This file contains hidden or 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
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.