-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
Changed gender choice types to color #29786
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
Closed
Closed
Conversation
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
Contributor
aaronsomi
commented
Jan 5, 2019
Q | A |
---|---|
Branch? | 3.4 |
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | #29737 |
License | MIT |
Doc PR | - |
* 4.2: Update PR template Fixes sprintf(): Too few arguments in Translator [Cache] fix deps updated CHANGELOG for 4.2.0-RC1 bumped Symfony version to 4.1.9 updated VERSION for 4.1.8 updated CHANGELOG for 4.1.8 bumped Symfony version to 3.4.20 updated VERSION for 3.4.19 updated CHANGELOG for 3.4.19 [Console] Move back root exception to stack trace in verbose mode
* 3.4: [DI] fix combinatorial explosion when analyzing the service graph [Debug] workaround opcache bug mutating "$this" !?!
* 4.1: [DI] fix combinatorial explosion when analyzing the service graph [Debug] workaround opcache bug mutating "$this" !?!
* 4.2: [DI] fix combinatorial explosion when analyzing the service graph [Debug] workaround opcache bug mutating "$this" !?!
* 3.4: calculate cache keys for property setters depending on the value updated VERSION for 2.8.48 update CONTRIBUTORS for 2.8.48 updated CHANGELOG for 2.8.48
This PR was merged into the 4.1 branch. Discussion ---------- [Routing] fix trailing slash redirection | Q | A | ------------- | --- | Branch? | 4.1 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29363 | License | MIT | Doc PR | - Commits ------- fbaba23 [Routing] fix trailing slash redirection
* 4.1: [Routing] fix trailing slash redirection calculate cache keys for property setters depending on the value updated VERSION for 2.8.48 update CONTRIBUTORS for 2.8.48 updated CHANGELOG for 2.8.48
* 4.2: [Routing] fix trailing slash redirection calculate cache keys for property setters depending on the value updated VERSION for 2.8.48 update CONTRIBUTORS for 2.8.48 updated CHANGELOG for 2.8.48
…ogizanagi) This PR was merged into the 4.2 branch. Discussion ---------- [Messenger] Mention HandleTrait in UPGRADE-4.2.md file | Q | A | ------------- | --- | Branch? | 4.2 <!-- see below --> | Bug fix? | no | New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | N/A <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | N/A Mentions this trait as a solution to build query buses. I've removed the callable suggestion, as I suspect this won't be the most useful one. Commits ------- 3cd4477 [Messenger] Mention HandleTrait in UPGRADE-4.2.md file
* 3.4: typo add a test case [Form] Handle all case variants of "nan" when parsing a number
* 4.1: typo add a test case [Form] Handle all case variants of "nan" when parsing a number
* 4.2: typo [Messenger] Mention HandleTrait in UPGRADE-4.2.md file add a test case [Form] Handle all case variants of "nan" when parsing a number
released v4.2.0
This PR was merged into the 4.1 branch. Discussion ---------- [Routing] fix greediness of trailing slash | Q | A | ------------- | --- | Branch? | 4.1 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29363 | License | MIT | Doc PR | - Commits ------- eb6aee6 [Routing] fix greediness of trailing slash
* 4.1: [Routing] fix greediness of trailing slash
* 4.2: [Routing] fix greediness of trailing slash bumped Symfony version to 4.2.1 updated VERSION for 4.2.0 updated CHANGELOG for 4.2.0
…deprecation warnings
This PR was merged into the 4.1 branch. Discussion ---------- [WebProfilerBundle] Fix title case | Q | A | ------------- | --- | Branch? | 4.1 | Bug fix? | yes | New feature? | no | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | #... <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | symfony/symfony-docs#... <!-- required for new features --> Split from #29312 for 4.1 👼 Commits ------- 3e16e25 [WebProfilerBundle] Fix title case
…erializer default id (ogizanagi) This PR was merged into the 4.2 branch. Discussion ---------- [FrameworkBundle][Messenger] Restore check for messenger serializer default id | Q | A | ------------- | --- | Branch? | 4.2 <!-- see below --> | Bug fix? | yes | New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | https://symfony-devs.slack.com/archives/C9PQ75TV3/p1543590611003500 <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | N/A From Slack: > @adamquaile [4:10 PM] > So, I just updated to 4.2 today, and got this message: > > The default Messenger serializer cannot be enabled as the Serializer support is not available. Try enabling it or running "composer require symfony/serializer-pack" > > In the docs it's stated: > > In order to use Symfony's built-in AMQP transport, you will need the Serializer Component. Ensure that it is installed with: > >But I haven't yet configured AMQP - I'm using my own transport. Should I be getting this exception? --- This check was removed in #28405, but is actually still necessary to not fail as soon as you can install the Messenger component without the Serializer one installed. Commits ------- 1cf17c0 [FrameworkBundle][Messenger] Restore check for messenger serializer default id
…ode) (redecs) This PR was squashed before being merged into the 4.3-dev branch (closes #29283). Discussion ---------- [Serializer] CsvEncoder no header option (encode / decode) | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #27447 | License | MIT | Doc PR | - This PR wants to introduce a new context option for the CsvEncoder, `CsvEncoder::NO_HEADERS_KEY` (boolean), that allows CSV encoding/decoding when you don't have/need a header. By default this is assumed to be false, so the headers are included in the CSV output or assumed to be present in the CSV input. When the option is set to true, the following behaviour occurs. Encoding === The following PHP input ```php array(array('a','b'), array('c', 'd')) ``` will generate this CSV output ```csv a,b c,d ``` Decoding === Considering the CSV input to be ```csv a,b c,d ``` the following PHP array will be returned ```php array ( 0 => array ( 0 => 'a', 1 => 'b', ), 1 => array ( 0 => 'c', 1 => 'd', ), ) ``` Commits ------- 0e63c61 [Serializer] CsvEncoder no header option (encode / decode)
Remove racially charged terms.
…ter) This PR was submitted for the master branch but it was merged into the 4.2 branch instead (closes #29744). Discussion ---------- Replace slave and master by replica and primary | Q | A | ------------- | --- | Branch? | 4.2 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29735 | License | MIT | Doc PR | I targeted 4.2 branch because the words were introduced in it cf. #27291 Commits ------- f06e6b4 Replace slave and master by replica and primary
… kernel (xabbuh) This PR was submitted for the master branch but it was merged into the 4.1 branch instead (closes #29708). Discussion ---------- [FrameworkBundle] access the container getting it from the kernel | Q | A | ------------- | --- | Branch? | 4.1 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29707 | License | MIT | Doc PR | Commits ------- 68ea3f1 access the container getting it from the kernel
This PR was merged into the 4.3-dev branch. Discussion ---------- [Validator] Choices constraint improvement | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | License | MIT Hi, here is a little improvement for the choice constraint: expose a new `choices` wildcard for the messages, in order to provide a way to display easily the valid choices to the user. thanks. Commits ------- 71dfa35 add new `choices` wildcard in message
…tener (thewilkybarkid) This PR was merged into the 4.3-dev branch. Discussion ---------- [HttpKernel] Increase priority of AddRequestFormatsListener | Q | A | ------------- | --- | Branch? | 2.8 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #19469 | License | MIT | Doc PR | Currently `AddRequestFormatsListener` has a low priority, so it won't fire before others like `RouterListener` which can create problems (eg when listening for a HTTP exception thrown by the router you don't have access for any custom types). IMO this map should be in the application rather than set on every request, but the same can be achieved by giving it a high priority. (Can't think of a reason for it to not be first...) Commits ------- 9bf3136 Increase priority of AddRequestFormatsListener
…kid) This PR was merged into the 4.3-dev branch. Discussion ---------- [HttpKernel] Set the default locale early | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Similar to #29186, the default locale isn't set till after the router so isn't available when trying to handle errors there (well, only the _default_ default locale is). Commits ------- 02c9f35 Set the default locale early
… (lyrixx) This PR was merged into the 4.3-dev branch. Discussion ---------- [FrameworkBundle] Added `ControllerTrait::isFormValid` | Q | A | ------------- | --- | Branch? | master (4.1) | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | symfony/symfony-docs#8518 Commits ------- 1a8c844 [FrameworkBundle] Added `ControllerTrait::isFormValid`
… rendered inside templates
… of dump() when rendered inside templates (dem3trio) This PR was squashed before being merged into the 4.3-dev branch (closes #29528). Discussion ---------- [DebugBundle] Added 'theme' option to change the color of dump() when rendered inside templates | Q | A | ------------- | --- | Branch? | master for features | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | License | MIT Added a html_dumper_theme option config to use the new HtmlDumper light theme when using dump() inside templates #SymfonyConHackday2018 Commits ------- 91e8057 [DebugBundle] Added 'theme' option to change the color of dump() when rendered inside templates
* 3.4: Fix: Adjust DocBlock \"ParserTest->getParserTestData()\" -> only some more tests [Lock] Pedantic improvements for lock [EventDispatcher] Fixed phpdoc on interface update year in license files [Console] Fix help text for single command applications Fix random test failure on lock improve error message when using test client without the BrowserKit component [Event Dispatcher] fixed 29703: TraceableEventDispatcher reset now sets callStack to null with test to dispatch after reset. Fixed minor typos Fix: Method can also return null [Stopwatch] Fixed phpdoc for category name
* 4.1: Fix: Adjust DocBlock \"ParserTest->getParserTestData()\" -> only some more tests access the container getting it from the kernel [Lock] Pedantic improvements for lock [EventDispatcher] Fixed phpdoc on interface update year in license files [Console] Fix help text for single command applications Fix random test failure on lock improve error message when using test client without the BrowserKit component [Event Dispatcher] fixed 29703: TraceableEventDispatcher reset now sets callStack to null with test to dispatch after reset. Fixed minor typos Fix: Method can also return null [Stopwatch] Fixed phpdoc for category name
* 4.2: update years in license files Fix: Adjust DocBlock \"ParserTest->getParserTestData()\" -> only some more tests access the container getting it from the kernel Replace slave and master by replica and primary Fix erasing cookies issue [Lock] Pedantic improvements for lock [EventDispatcher] Fixed phpdoc on interface update year in license files [VarExporter] fix exporting array indexes [SecurityBundle] Fix traceable voters [Console] Fix help text for single command applications Fix random test failure on lock improve error message when using test client without the BrowserKit component Fixed minor typos in an error message [Event Dispatcher] fixed 29703: TraceableEventDispatcher reset now sets callStack to null with test to dispatch after reset. Fixed minor typos Fix: Method can also return null [Stopwatch] Fixed phpdoc for category name
It this happens, and it happens, this will make the build quite slow. Refs #29760
…ereguiluz) This PR was squashed before being merged into the 4.3-dev branch (closes #29755). Discussion ---------- Improved BIC + IBAN validator for some special cases | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | - | License | MIT | Doc PR | - While blogging about [Improved Financial Constraints in Symfony 4.3](https://symfony.com/blog/new-in-symfony-4-3-improved-financial-constraints) I found this resource that mentions some exceptions when validating BIC + IBAN: https://wiki.xmldation.com/index.php?title=Support/EPC/IBAN_%26_BIC_relation I don't know if that resource is fully reliable ... but I checked it in the official "European Banking Resources" website (https://www.ecbs.org/iban/) and most of it matches, so I think we can safely make this change. Commits ------- 88a1696 Improved BIC + IBAN validator for some special cases
…ds (greg0ire) This PR was merged into the 4.2 branch. Discussion ---------- [PHPUnitBridge][TestSuite] Do not risk waiting 100 seconds | Q | A | ------------- | --- | Branch? | 4.2 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | nofiles --> | Tests pass? | yes, a bit more often | Fixed tickets | not entirely fixing it | License | MIT | Doc PR | n/a It this happens, and it happens, this will make the build quite slow. Refs #29760 Commits ------- 9f45af4 Do not risk waiting 100 seconds
…(ro0NL)" (chalasr) This PR was merged into the 4.2 branch. Discussion ---------- Revert "minor #28610 [Form] Check for Intl availibility (ro0NL)" | Q | A | ------------- | --- | Branch? | 4.2 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a Intl is an hard dependency for Form, these checks cannot be reached. Commits ------- 3ac98a6 Revert "minor #28610 [Form] Check for Intl availibility (ro0NL)"
…eming (yceruto) This PR was merged into the 4.3-dev branch. Discussion ---------- [Form] Add new block_prefix option for an easy form theming | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29651 | License | MIT | Doc PR | symfony/symfony-docs#10835 Commits ------- bd50ac4 Add block_prefix option for an easy form theming
* 3.4: Ensure final input of CommandTester works with default [Intl] handle null date and time types Do not ignore the choice groups for caching
* 4.1: Ensure final input of CommandTester works with default [Intl] handle null date and time types Do not ignore the choice groups for caching
* 4.2: Ensure final input of CommandTester works with default Do not risk waiting 100 seconds [Intl] handle null date and time types Revert "minor #28610 [Form] Check for Intl availibility (ro0NL)" Do not ignore the choice groups for caching
Signed-off-by: Aaron Somi <aaronsomi@gmail.com>
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.