Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Form] Add input withstring value inMoneyType#59993

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

StevenRenaux
Copy link
Contributor

@StevenRenauxStevenRenaux commentedMar 17, 2025•
edited by xabbuh
Loading

QA
Branch?7.3
Bug fix?no
New feature?yes
Deprecations?no
IssuesFix#59710
LicenseMIT
Doc PR#20768

Related to the issue, add the possibility to configure the propertyinput with the valuestring to avoid some unnecessary update with Doctrine.

@carsonbotcarsonbot added this to the7.2 milestoneMar 17, 2025
@StevenRenauxStevenRenauxforce-pushed thefeatures/Allow-string-input-for-MoneyType branch from99fa4f2 todcd501cCompareMarch 17, 2025 12:41
@StevenRenauxStevenRenaux changed the titleAdd input withstring value inMoneyType[Form] Add input withstring value inMoneyTypeMar 17, 2025
@xabbuhxabbuh changed the base branch from7.2 to7.3March 18, 2025 07:43
@xabbuhxabbuh modified the milestones:7.2,7.3Mar 18, 2025
@xabbuhxabbuh removed the Bug labelMar 18, 2025
@carsonbotcarsonbot changed the title[Form] Add input withstring value inMoneyTypeAdd input withstring value inMoneyTypeMar 18, 2025
@xabbuhxabbuh added the Form labelMar 18, 2025
@carsonbotcarsonbot changed the titleAdd input withstring value inMoneyType[Form] Add input withstring value inMoneyTypeMar 18, 2025
@StevenRenauxStevenRenauxforce-pushed thefeatures/Allow-string-input-for-MoneyType branch 2 times, most recently from190e1aa to5ab085dCompareMarch 19, 2025 09:21
@StevenRenauxStevenRenauxforce-pushed thefeatures/Allow-string-input-for-MoneyType branch from5ab085d to612024eCompareMarch 20, 2025 16:09
Copy link
Member

@xabbuhxabbuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Looks very good. Just a minor suggestion for rewording the changelog message.

StevenRenaux reacted with thumbs up emoji
@StevenRenauxStevenRenauxforce-pushed thefeatures/Allow-string-input-for-MoneyType branch from612024e tob5d6bf8CompareMarch 21, 2025 08:13
@fabpot
Copy link
Member

Thank you@StevenRenaux.

@fabpotfabpot merged commit272c580 intosymfony:7.3Mar 21, 2025
10 of 11 checks passed
OskarStark added a commit to symfony/symfony-docs that referenced this pull requestApr 7, 2025
…naux)This PR was merged into the 7.3 branch.Discussion----------[Form] Update `input` option for `MoneyType`Related tosymfony/symfony#59993Commits-------be54b38 Update input option for MoneyType
@fabpotfabpot mentioned this pull requestMay 2, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@fabpotfabpotfabpot approved these changes

@xabbuhxabbuhxabbuh approved these changes

@ycerutoycerutoyceruto approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
7.3
Development

Successfully merging this pull request may close these issues.

[Form] MoneyType: allow "string" as input
5 participants
@StevenRenaux@fabpot@xabbuh@yceruto@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp