Skip to content

Update AbstractBackend.php #12120

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
Nov 23, 2017
Merged

Update AbstractBackend.php #12120

merged 1 commit into from
Nov 23, 2017

Conversation

hewersonfreitas
Copy link

Hello guys, when the validation message is returned, the attribute code is displayed.

That prevents a better translation for other languages, has been adjusted to return the label instead of the attribute code.

Adjustments on lines 234, 241, 254.

Description

Fixed Issues (if relevant)

  1. magento/magento2#<issue_number>: Issue title
  2. ...

Manual testing scenarios

  1. ...
  2. ...

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

Hello guys, when the validation message is returned, the attribute code is displayed.

That prevents a better translation for other languages, has been adjusted to return the label instead of the attribute code.

Adjustments on lines 234, 241, 254.
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Nov 8, 2017

CLA assistant check
All committers have signed the CLA.

@ihor-sviziev
Copy link
Contributor

ihor-sviziev commented Nov 10, 2017

@hewersonfreitas tests failed. Could you adjust your code or tests (if needed)?
Could you also add steps to reproduce? It will help to test your change, and process your PR faster.

@okorshenko okorshenko self-assigned this Nov 21, 2017
@okorshenko okorshenko added this to the November 2017 milestone Nov 21, 2017
@okorshenko okorshenko self-assigned this Nov 21, 2017
@okorshenko okorshenko merged commit dd40f2b into magento:2.2-develop Nov 23, 2017
okorshenko pushed a commit that referenced this pull request Nov 23, 2017
okorshenko pushed a commit that referenced this pull request Nov 23, 2017
magento-engcom-team pushed a commit that referenced this pull request Feb 8, 2018
@hewersonfreitas hewersonfreitas deleted the patch-1 branch January 21, 2020 19:15
@hewersonfreitas
Copy link
Author

I accidentally deleted hewersonfreitas: patch-1, could it revert?

I can't display on my github that you contribute to the project

@ihor-sviziev
Copy link
Contributor

@hewersonfreitas pr is accepted and merged, you could safely remove branch

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.

5 participants