Skip to content
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

Bug fix for removing keys from validation.attributes in translation managers #1861

Merged
merged 5 commits into from
Sep 25, 2021
Merged

Bug fix for removing keys from validation.attributes in translation managers #1861

merged 5 commits into from
Sep 25, 2021

Conversation

andrey-helldar
Copy link
Member

Problem

Translation managers don't carry keys from an attribute in an app on update.

Reason

The transfer is done by reading the keys. There are no attributes keys in the source/validation.php and source/validation-inline.php files.

Solution

Add keys to the source files.

source/validation.php Show resolved Hide resolved
@caouecs
Copy link
Member

caouecs commented Sep 25, 2021

I can merge ?

@andrey-helldar
Copy link
Member Author

@caouecs, yes

@caouecs caouecs merged commit 307c3f8 into Laravel-Lang:master Sep 25, 2021
@caouecs
Copy link
Member

caouecs commented Sep 25, 2021

done

@andrey-helldar
Copy link
Member Author

Thank you 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants