Skip to content

refactor(forms): change control suffix to fieldff#59

Merged
1 commit merged intoalphafrom
rename-control-to-field
May 6, 2020
Merged

refactor(forms): change control suffix to fieldff#59
1 commit merged intoalphafrom
rename-control-to-field

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented May 5, 2020

Todo:

  • Rename *Control components in the forms workspace to *FieldFF (e. g. InputField in widgets; InputFieldFF in forms)
  • Rename FieldSetField to FieldGroup in widgets and GroupControl to FieldGroupFF

@cypress
Copy link
Copy Markdown

cypress bot commented May 5, 2020



Test summary

423 0 0 0


Run details

Project ui
Status Passed
Commit 7479f61
Started May 6, 2020 9:12 AM
Ended May 6, 2020 9:20 AM
Duration 08:06 💡
OS Linux Ubuntu Linux - 18.04
Browser Electron 80

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@ghost ghost mentioned this pull request May 5, 2020
Merged
15 tasks
@ghost ghost marked this pull request as ready for review May 5, 2020 13:27
@ghost ghost self-requested a review as a code owner May 5, 2020 13:27
Copy link
Copy Markdown
Contributor

@Mohammer5 Mohammer5 left a comment

Choose a reason for hiding this comment

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

Only found some file path issues. LGTM

Comment thread cypress/integration/forms/SingleSelect/common/index.js
Comment thread cypress/integration/forms/Switch/can_toggle_a_boolean/index.js
Comment thread cypress/integration/forms/Switch/can_toggle_a_value/index.js
Comment thread cypress/integration/forms/Switch/displays_error/index.js
@ghost
Copy link
Copy Markdown
Author

ghost commented May 5, 2020

@Mohammer5 I've updated the paths (54261de), that's what you meant right?

Copy link
Copy Markdown
Contributor

@Mohammer5 Mohammer5 left a comment

Choose a reason for hiding this comment

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

yes

@ghost ghost force-pushed the rename-control-to-field branch from 54261de to a896a17 Compare May 6, 2020 09:08
We've renamed our final-form enabled components and some related
components to make our naming more uniform. The following changes have
been made:

- For final-form enabled components the Control suffix has been changed
  to FieldFF. The FF stands for final-form, clarifying that the
  component is tied to final-form and making the relation with our
  regular Field components more clear. So, for example, instead of
  <InputControl /> we now have <InputFieldFF />
- The <FieldSetField /> component has been renamed to <FieldGroup />
- The <GroupControl /> component has been renamed to <FieldGroupFF />
@ghost ghost force-pushed the rename-control-to-field branch from a896a17 to 7479f61 Compare May 6, 2020 09:10
@ghost ghost merged commit 64e9b4c into alpha May 6, 2020
@ghost ghost deleted the rename-control-to-field branch May 6, 2020 09:31
@dhis2-bot
Copy link
Copy Markdown
Contributor

@dhis2-bot
Copy link
Copy Markdown
Contributor

This pull request was closed.
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