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

feat(textInput,formHelpers): discovery-151 pf4 text input #149

Merged
merged 2 commits into from
Aug 31, 2022

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Aug 30, 2022

What's included

Features

  1. textInput,formHelpers discovery-151 pf4 text input

Bug Fixes

  1. dropdownSelect discovery-151 undefined options

Notes

  • extracted from the toolbar work
  • prep for form updates to pf4

How to test

Coverage and basic unit test check

  1. update the NPM packages with $ yarn
  2. $ yarn test
  3. confirm tests come back clean

Local run check

  1. update the NPM packages with $ yarn
  2. $ yarn start:stage
  3. confirm interaction with existing dropdownSelect fields remains unchanged

Example

...

Updates issue/story

DISCOVERY-151
DISCOVERY-153
#104

@cdcabrera cdcabrera added the pf4 Tracking for pf4 refactors label Aug 30, 2022
@cdcabrera cdcabrera requested a review from ruda August 30, 2022 02:43
@cdcabrera cdcabrera changed the title feat(formFieldTextInput): discovery-151 pf4 text input feat(textInput,formHelpers): discovery-151 pf4 text input Aug 30, 2022
@cdcabrera cdcabrera changed the base branch from dev to dev-forms August 31, 2022 16:06
cdcabrera added a commit to cdcabrera/quipucords-ui that referenced this pull request Aug 31, 2022
…#149)

* textInput, pf4 text input wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit to cdcabrera/quipucords-ui that referenced this pull request Aug 31, 2022
* options, undefined check
* activateOptions, length check
@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2022

Codecov Report

❗ No coverage uploaded for pull request base (dev-forms@78d8c15). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             dev-forms     #149   +/-   ##
============================================
  Coverage             ?   77.41%           
============================================
  Files                ?      105           
  Lines                ?     3392           
  Branches             ?     1016           
============================================
  Hits                 ?     2626           
  Misses               ?      671           
  Partials             ?       95           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78d8c15...be59628. Read the comment docs.

…#149)

* textInput, pf4 text input wrapper
* formHelpers, mock event, future validation
* options, undefined check
* activateOptions, length check
@cdcabrera
Copy link
Member Author

cdcabrera commented Aug 31, 2022

As of 20220831 after review, since this contains toolbar functionality/fixes we need to merge this into dev then sync with the forms feature branch

@cdcabrera cdcabrera changed the base branch from dev-forms to dev August 31, 2022 18:48
@cdcabrera cdcabrera merged commit 0a6c012 into quipucords:dev Aug 31, 2022
cdcabrera added a commit that referenced this pull request Aug 31, 2022
* textInput, pf4 text input wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit that referenced this pull request Aug 31, 2022
* textInput, pf4 textInput wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit that referenced this pull request Aug 31, 2022
* options, undefined check
* activateOptions, length check
cdcabrera added a commit that referenced this pull request Sep 7, 2022
* textInput, pf4 textInput wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit that referenced this pull request Sep 7, 2022
* options, undefined check
* activateOptions, length check
cdcabrera added a commit that referenced this pull request Sep 8, 2022
* textInput, pf4 textInput wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit that referenced this pull request Sep 8, 2022
* options, undefined check
* activateOptions, length check
cdcabrera added a commit that referenced this pull request Sep 20, 2022
* textInput, pf4 textInput wrapper
* formHelpers, mock event, future validation
cdcabrera added a commit that referenced this pull request Sep 20, 2022
* options, undefined check
* activateOptions, length check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pf4: forms pf4 Tracking for pf4 refactors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants