Skip to content

Conversation

anzin
Copy link
Contributor

@anzin anzin commented Feb 16, 2021

Description (*)
I've fixed the generator Data Provider Class and Ui component form.
 
image

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)

Copy link
Contributor

@eduard13 eduard13 left a comment

Choose a reason for hiding this comment

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

Hi @anzin, could you please check the below clarifications?

Thank you.

Copy link
Contributor

@eduard13 eduard13 left a comment

Choose a reason for hiding this comment

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

In order to make the code cleaner, could you please review the below suggestions?

Please let me know if they are wrong or you have any questions regarding them.

Thank you.

@eduard13 eduard13 self-requested a review February 17, 2021 13:47
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