Skip to content

Updated default validation mapping directory #12982

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 1 commit into from
Feb 19, 2020

Conversation

zairigimad
Copy link
Contributor

Change the default mapping directory

Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

No need to adjust the xml example?
Sorry I am currently on a phone and cannot double check it 👍🏻

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

Thanks for that PR, the other formats below (XML and PHP) need indeed to be updated as well

@HeahDude HeahDude added this to the 4.4 milestone Feb 19, 2020
@HeahDude HeahDude changed the title default validation mapping directory Updated default validation mapping directory Feb 19, 2020
@zairigimad
Copy link
Contributor Author

Thanks for that PR, the other formats below (XML and PHP) need indeed to be updated as well

@HeahDude done

@OskarStark OskarStark changed the base branch from 5.0 to 4.4 February 19, 2020 10:03
@OskarStark
Copy link
Contributor

Thank you Zairig.

OskarStark added a commit that referenced this pull request Feb 19, 2020
This PR was submitted for the 5.0 branch but it was squashed and merged into the 4.4 branch instead.

Discussion
----------

Updated default validation mapping directory

Change the default mapping directory

Commits
-------

f538ce9 Updated default validation mapping directory
@OskarStark OskarStark merged commit f538ce9 into symfony:4.4 Feb 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants