Skip to content

chore: merge 2.6 #1476

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

Merged
merged 39 commits into from
Nov 24, 2021
Merged

chore: merge 2.6 #1476

merged 39 commits into from
Nov 24, 2021

Conversation

alanpoulain
Copy link
Member

No description provided.

kochen and others added 30 commits September 1, 2021 13:33
…1425)

Co-authored-by: Jan Christoph Beyer <jbe@plusnet.de>
It is better to use `Type::BUILTIN_TYPE_STRING` in the documentation. 
The reasons for it are: 
- It's a Symfony thing
- By using Symfony Property Info Type the manual conveys the right thinking direction. The custom filter wont work if you use `'boolean'` as the type, the valid value for boolean is `'bool'` which is the value of the constant `Type::BUILTIN_TYPE_BOOL`, 
- By using `Type::BUILTIN_TYPE_STRING` in the documentation, developers will be pushed to the right thinking direction when building a custom filter
* docs: Remove empty string defaults from entities

I'm not getting the SQL exception in 'Validating Data' section; empty strings are being stored instead of null being attempted.

The simplest fix is to remove the default empty strings from the entities.

This is using the Symfony and Composer install.

* Update index.md

* Update index.md

* Reinstate default empty strings where removed.

* Add unvalidated title explanation

* Add NotBlank constraint to title
* Update jwt.md for Symfony 5.3 compatibility

update doc for Symfony 5.3 compatibility, due to use deprecated code app does not work and return warnings

- New in version 5.3: The password_hashers option was introduced in Symfony 5.3. In previous versions it was called encoders.
- Deprecated since version 5.3: Guard authenticators are deprecated since Symfony 5.3 in favor of the new authenticator-based system.

Reference: 
https://symfony.com/doc/current/security.html#c-hashing-passwords
https://symfony.com/doc/current/security.html#guard-authenticators

* Apply suggestions from code review

Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>

* docs: use PUBLIC_ACCESS

Co-authored-by: Alan Poulain <contact@alanpoulain.eu>
Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>
* docs: Change annotations to attribute in core and admin pages

* doc: fix typos and brackets

Apply suggestions from code review

Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>

* doc: removed attribute key

* docs: Change annotations to attribute in core and admin pages

* doc: fix typos and brackets

Apply suggestions from code review

Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>

* doc: removed attribute key

Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>
The anonymous option for the firewall config was removed in symfony 5.1
as explained here: symfony/symfony#36574

Using the option caused an error but it can be skipped. "[A] user that
is not authenticated is similar to an "unknown" user: They both have no
rights at all."

Resolves api-platform#1445
…pose

fix: deployment with Docker Compose
ker0x and others added 9 commits November 2, 2021 17:15
* docs: add guide to migrate from FOSRestBundle

* Fix typos

Co-authored-by: Alan Poulain <contact@alanpoulain.eu>

* Fix reviews

* Fix reviews

* Add migration guides to outline.yaml

* Fix typos

* Reformat the page, fix the menu

* Remove paragraph about FOSRest's deprecation

* Adding links to the page, moving the new section below Configuration

Co-authored-by: Alan Poulain <contact@alanpoulain.eu>
@alanpoulain alanpoulain changed the title Merge 2.6 chore: merge 2.6 Nov 24, 2021
@alanpoulain alanpoulain merged commit 61e8197 into api-platform:main Nov 24, 2021
@alanpoulain alanpoulain deleted the merge-2.6 branch November 24, 2021 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.