Skip to content

Conversation

@drpayyne
Copy link
Contributor

@drpayyne drpayyne commented Oct 22, 2020

Description

This PR changes the validator for com.magento.idea.magento2plugin.actions.generation.dialog.InjectAViewModelDialog and fixes some grammar

Fixed Issues

  1. Change validation for the injection of new ViewModel dialog window #294: Change validation for the injection of new ViewModel dialog window

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 integration/functional tests (if applicable)
  • All automated tests passed successfully (all builds are green)

@coderimus coderimus self-assigned this Oct 23, 2020
@coderimus coderimus self-requested a review October 23, 2020 07:46
@coderimus coderimus merged commit 47e7479 into magento:2.1.0-develop Oct 23, 2020
common.name=Name
common.className=Class Name
common.className=Class name
common.argument=Argument name
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @VitaliyBoyko, @coderimus. I was just doing some local testing and I noticed that this line (line 26) with the value for common.argument is missing from the current codebase? This PR was merged but this particular file doesn't seem to be 'registered' in the git commit history? I'm very confused, maybe I'm missing something. Could you please check on this? Thanks.

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.

3 participants