feat(modal): discovery-148 pf4 modal wrapper #110

cdcabrera · 2022-06-30T20:38:47Z

What's included

Chores

build activate eslint jsdoc (9c3aa91)

Bug Fixes

pf4-style expose scss variables (e402a77)

Features

modal discovery-149 pf4 modal wrapper (1feaa58)

Notes

Build updates
SCSS variables applied
Modal wrapper created for future modal needs, see refactor(modal): discovery-148 pf4 modal wrapper #88
There are NO visual/design updates associated with this request

How to test

Coverage and basic unit test check

update the NPM packages with $ yarn
$ yarn test
confirm tests come back clean

Check the build

update the NPM packages with $ yarn
$ yarn build
confirm the build completes without error

Example

...

Updates issue/story

DISCOVERY-148
related #88
Related #101

src/components/modal/__tests__/__snapshots__/modal.test.js.snap

src/components/modal/__tests__/modal.test.js

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

codecov-commenter · 2022-06-30T21:12:50Z

Codecov Report

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

❗ Current head d9ef060 differs from pull request most recent head d6710cf. Consider uploading reports for the commit d6710cf to get more accurate results

@@          Coverage Diff           @@
##             dev     #110   +/-   ##
======================================
  Coverage       ?   72.03%           
======================================
  Files          ?       95           
  Lines          ?     3040           
  Branches       ?      823           
======================================
  Hits           ?     2190           
  Misses         ?      752           
  Partials       ?       98

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 d9c207e...d6710cf. Read the comment docs.

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

author CD Cabrera <[email protected]> 1654528943 -0400 committer jenny-s51 <[email protected]> 1659366125 -0400 fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) refactor(services): discovery-8 minor config restructure (quipucords#82) * service config, allow opts, fallback xhr timeout * api calls, config wrapper function, named imports refactor(redux): discovery-8 middleware, hooks, restructure (quipucords#82) * helpers, aggregatedError func for redux hooks * redux actions, named exports * redux hooks, useReactRedux * redux middleware, middleware export, multi-actions * store, move middleware, named export fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) refactor(authentication): discovery-148 pf4 alert, locale (quipucords#112) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(refreshTimeButton): discovery-149 pf4 button, icon (quipucords#114) refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state refactor(toastNotificationsList): discovery-148, pf4 alert (quipucords#115) * build, dotenv toast timeout * components, adjust alertType to alert variant * helpers, expose toast timeout * toastNotificationsList, pf4 alert, alertGroup * toastNotificationsReducer, adjust pause, resume fix(testing): expand toastNotificationsList snapshots (quipucords#115) refactor(sources): discovery-148 pf4 empty-state (quipucords#120) * locale, add empty-state strings * sources, pf4 empty-state refactor(creds,sources,scans): discovery-149 emptyState grid (quipucords#121) * credentials, sources, scansEmptyState, remove grid fix(i18n): discovery-8 partial context into keys (quipucords#124) feat(wizard): discovery-153 pf4 wizard wrapper (quipucords#124) * confirmationModal, pass isContentOnly * modal, add isContentOnly, removes padding * wizard, consistent pf4 wrapper, allow modal sizing refactor(addSourceWizard): discovery-153, pf4 wizard (quipucords#124) * locale, wizard form strings * addSourceWizard, Constants, restructure steps * addSourceWizardStepThree, add empty-state * styling, restructure pf3 styling wizard form refactor(scanHostList): discovery-148, pf4 empty-state (quipucords#125) refactor(pagination): wip refactor(pagination): use PF4 component refactor(pagination): update snapshots refactor(pagination): update test refactor(pagination): update test refactor(creds,sources,scans): discovery-149 pf4 grid, list (quipucords#126) * credentialsListItem, grid to list, tooltip * credentials, removed grid * scanHostList, grid to list * scanSourceList nested list with icon * sourceCredentialsList, list with icon refactor(dropdownSelect): discovery-152 pf4 select, dropdown (quipucords#127) * addSourceWizardStepTwo, options, pf4 select list * createCredentialDialog, pf4 select list * dropdownSelect, use Curiosity pf4 select wrapper * locale, option strings refactor(dropdownSelect): discovery-152 pf4 dropdown (quipucords#130) feat(addCredentialType): discovery-152 button selector (quipucords#130) refactor(credentials): discovery-152, pf4 dropdown (quipucords#130) * credentials, pf4 button, addCredentialType * credentialsEmptyState, addCredentialType * locale, cred strings

author CD Cabrera <[email protected]> 1654528943 -0400 committer jenny-s51 <[email protected]> 1659366125 -0400 fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) refactor(services): discovery-8 minor config restructure (quipucords#82) * service config, allow opts, fallback xhr timeout * api calls, config wrapper function, named imports refactor(redux): discovery-8 middleware, hooks, restructure (quipucords#82) * helpers, aggregatedError func for redux hooks * redux actions, named exports * redux hooks, useReactRedux * redux middleware, middleware export, multi-actions * store, move middleware, named export fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) refactor(authentication): discovery-148 pf4 alert, locale (quipucords#112) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(refreshTimeButton): discovery-149 pf4 button, icon (quipucords#114) refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state refactor(toastNotificationsList): discovery-148, pf4 alert (quipucords#115) * build, dotenv toast timeout * components, adjust alertType to alert variant * helpers, expose toast timeout * toastNotificationsList, pf4 alert, alertGroup * toastNotificationsReducer, adjust pause, resume fix(testing): expand toastNotificationsList snapshots (quipucords#115) refactor(sources): discovery-148 pf4 empty-state (quipucords#120) * locale, add empty-state strings * sources, pf4 empty-state refactor(creds,sources,scans): discovery-149 emptyState grid (quipucords#121) * credentials, sources, scansEmptyState, remove grid fix(i18n): discovery-8 partial context into keys (quipucords#124) feat(wizard): discovery-153 pf4 wizard wrapper (quipucords#124) * confirmationModal, pass isContentOnly * modal, add isContentOnly, removes padding * wizard, consistent pf4 wrapper, allow modal sizing refactor(addSourceWizard): discovery-153, pf4 wizard (quipucords#124) * locale, wizard form strings * addSourceWizard, Constants, restructure steps * addSourceWizardStepThree, add empty-state * styling, restructure pf3 styling wizard form refactor(scanHostList): discovery-148, pf4 empty-state (quipucords#125) refactor(pagination): wip refactor(pagination): use PF4 component refactor(pagination): update snapshots refactor(pagination): update test refactor(pagination): update test refactor(creds,sources,scans): discovery-149 pf4 grid, list (quipucords#126) * credentialsListItem, grid to list, tooltip * credentials, removed grid * scanHostList, grid to list * scanSourceList nested list with icon * sourceCredentialsList, list with icon refactor(dropdownSelect): discovery-152 pf4 select, dropdown (quipucords#127) * addSourceWizardStepTwo, options, pf4 select list * createCredentialDialog, pf4 select list * dropdownSelect, use Curiosity pf4 select wrapper * locale, option strings refactor(dropdownSelect): discovery-152 pf4 dropdown (quipucords#130) feat(addCredentialType): discovery-152 button selector (quipucords#130) refactor(credentials): discovery-152, pf4 dropdown (quipucords#130) * credentials, pf4 button, addCredentialType * credentialsEmptyState, addCredentialType * locale, cred strings refactor(pagination): update snapshots

fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) refactor(services): discovery-8 minor config restructure (quipucords#82) * service config, allow opts, fallback xhr timeout * api calls, config wrapper function, named imports refactor(redux): discovery-8 middleware, hooks, restructure (quipucords#82) * helpers, aggregatedError func for redux hooks * redux actions, named exports * redux hooks, useReactRedux * redux middleware, middleware export, multi-actions * store, move middleware, named export fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) refactor(authentication): discovery-148 pf4 alert, locale (quipucords#112) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(refreshTimeButton): discovery-149 pf4 button, icon (quipucords#114) refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state fix(build): base dep package updates (quipucords#80) * build, base deps for dev functionality * build, stage, review, use pre-built image * testing, dist test, lint updates * styling, flip to sass, remove legacy css chore(build): node, workflow, docs, and tooling refs (quipucords#80) fix(build): npm updates (quipucords#80) fix(build): update react (quipucords#80) * build, update react 17, pf deps * testing, snapshots, console check fix(i18n): component loading (quipucords#80) chore(build): github badge, remove travis (quipucords#81) chore(build): post clean up, remove unused css copy (quipucords#84) fix(build): npm updates (quipucords#83) fix(build): npm update, redux middleware (quipucords#82) fix(build): discovery-8 allow repo locale files (quipucords#85) * docs, merge repo locale with docs locale files * helpers, expand noopTranslate, annotations * i18n, snapshot translate uses * i18nHelpers, translate helper, expand testing * testing, hook testing funcs fix(build): allow running review, stage parallel (quipucords#86) * build, parallel images, container names, domains * build, npm script, docker stop feat(pf4-style): discovery-8 pf4 base css, scss (quipucords#86) * build, pf4 npm updates * style, move base css files to index.scss * style, apply core pf4, overrides.scss fix(style): discovery-8 css, scss vars for color (quipucords#86) fix(testing): use i18n unit, remove integration check (quipucords#87) fix(helpers): discovery-8 consistent brand, getCurrentDate (quipucords#87) refactor(aboutModal): discovery-148 pf4 conversion (quipucords#87) * aboutModal, pf4 component conversion * locales, about modal strings chore(build): do not start with stored data, db volumes (quipucords#107) chore(build): activate eslint jsdoc (quipucords#110) fix(pf4-style): expose scss variables (quipucords#110) feat(modal): discovery-148 pf4 modal wrapper (quipucords#110) * locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable refactor(modal): discovery-148 pf4 modal wrapper (quipucords#88) * locale, button strings for modals * authentication, apply pf4 modal * confirmationModal, refactor using pf4 modal * createCredentialDialog, pf4 modal, buttons * createScanDialog, pf4 modal, buttons * credentials, pf4 modal * mergeReportsDialog, pf modal, buttons * scans, pf4 modal * sources, pf4 modal refactor(createCredentialDialog): discovery-148 pf4 alert(quipucords#108) fix(confirmationModal): expand behavior, allow children (quipucords#113) refactor(authentication): discovery-148 replace empty-state (quipucords#113) refactor(credentials): discovery-148 pf4 empty-state (quipucords#116) * locale, align empty-state strings * authentication, align with credentials * credentials, credentialsEmptyState, pf4 empty-state refactor(scans): discovery-148 pf4 empty-state (quipucords#117) * locale, align empty-state strings * scans, jobsList, sourceList, pf4 empty-state refactor(toastNotificationsList): discovery-148, pf4 alert (quipucords#115) * build, dotenv toast timeout * components, adjust alertType to alert variant * helpers, expose toast timeout * toastNotificationsList, pf4 alert, alertGroup * toastNotificationsReducer, adjust pause, resume fix(testing): expand toastNotificationsList snapshots (quipucords#115) refactor(sources): discovery-148 pf4 empty-state (quipucords#120) * locale, add empty-state strings * sources, pf4 empty-state refactor(creds,sources,scans): discovery-149 emptyState grid (quipucords#121) * credentials, sources, scansEmptyState, remove grid fix(i18n): discovery-8 partial context into keys (quipucords#124) feat(wizard): discovery-153 pf4 wizard wrapper (quipucords#124) * confirmationModal, pass isContentOnly * modal, add isContentOnly, removes padding * wizard, consistent pf4 wrapper, allow modal sizing refactor(addSourceWizard): discovery-153, pf4 wizard (quipucords#124) * locale, wizard form strings * addSourceWizard, Constants, restructure steps * addSourceWizardStepThree, add empty-state * styling, restructure pf3 styling wizard form refactor(scanHostList): discovery-148, pf4 empty-state (quipucords#125) refactor(pagination): wip refactor(pagination): use PF4 component refactor(pagination): update snapshots refactor(pagination): update test refactor(pagination): update test refactor(creds,sources,scans): discovery-149 pf4 grid, list (quipucords#126) * credentialsListItem, grid to list, tooltip * credentials, removed grid * scanHostList, grid to list * scanSourceList nested list with icon * sourceCredentialsList, list with icon refactor(dropdownSelect): discovery-152 pf4 select, dropdown (quipucords#127) * addSourceWizardStepTwo, options, pf4 select list * createCredentialDialog, pf4 select list * dropdownSelect, use Curiosity pf4 select wrapper * locale, option strings refactor(dropdownSelect): discovery-152 pf4 dropdown (quipucords#130) feat(addCredentialType): discovery-152 button selector (quipucords#130) refactor(credentials): discovery-152, pf4 dropdown (quipucords#130) * credentials, pf4 button, addCredentialType * credentialsEmptyState, addCredentialType * locale, cred strings refactor(pagination): update snapshots

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added the pf4 Tracking for pf4 refactors label Jun 30, 2022

cdcabrera commented Jun 30, 2022

View reviewed changes

src/components/modal/__tests__/__snapshots__/modal.test.js.snap Outdated Show resolved Hide resolved

src/components/modal/__tests__/modal.test.js Outdated Show resolved Hide resolved

cdcabrera force-pushed the dis-149-modal-wrapper branch from b911884 to d9ef060 Compare June 30, 2022 21:07

cdcabrera added 3 commits June 30, 2022 17:08

chore(build): activate eslint jsdoc (quipucords#110)

03c5512

fix(pf4-style): expose scss variables (quipucords#110)

b301ec6

feat(modal): discovery-149 pf4 modal wrapper (quipucords#110)

d6710cf

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera force-pushed the dis-149-modal-wrapper branch from d9ef060 to d6710cf Compare June 30, 2022 21:12

cdcabrera marked this pull request as ready for review June 30, 2022 21:12

cdcabrera merged commit 81865f4 into quipucords:dev Jun 30, 2022

cdcabrera added a commit that referenced this pull request Jun 30, 2022

chore(build): activate eslint jsdoc (#110)

2562441

cdcabrera added a commit that referenced this pull request Jun 30, 2022

fix(pf4-style): expose scss variables (#110)

83f7788

cdcabrera changed the title ~~feat(modal): discovery-149 pf4 modal wrapper~~ feat(modal): discovery-148 pf4 modal wrapper Jul 1, 2022

cdcabrera added a commit that referenced this pull request Jul 1, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

be15baa

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added a commit that referenced this pull request Jul 12, 2022

chore(build): activate eslint jsdoc (#110)

c729a5b

cdcabrera added a commit that referenced this pull request Jul 12, 2022

fix(pf4-style): expose scss variables (#110)

fe79f8d

cdcabrera added a commit that referenced this pull request Jul 12, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

ffbe803

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added the pf4:wrapper-component label Aug 12, 2022

cdcabrera added a commit that referenced this pull request Aug 12, 2022

chore(build): activate eslint jsdoc (#110)

2d91e6f

cdcabrera added a commit that referenced this pull request Aug 12, 2022

fix(pf4-style): expose scss variables (#110)

1d3f7ac

cdcabrera added a commit that referenced this pull request Aug 12, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

8956679

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added a commit that referenced this pull request Sep 7, 2022

chore(build): activate eslint jsdoc (#110)

fb0d289

cdcabrera added a commit that referenced this pull request Sep 7, 2022

fix(pf4-style): expose scss variables (#110)

290df66

cdcabrera added a commit that referenced this pull request Sep 7, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

c30556f

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added a commit that referenced this pull request Sep 8, 2022

chore(build): activate eslint jsdoc (#110)

f77d4fa

cdcabrera added a commit that referenced this pull request Sep 8, 2022

fix(pf4-style): expose scss variables (#110)

0ab8f5b

cdcabrera added a commit that referenced this pull request Sep 8, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

e5ba797

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

cdcabrera added a commit that referenced this pull request Sep 20, 2022

chore(build): activate eslint jsdoc (#110)

3d58297

cdcabrera added a commit that referenced this pull request Sep 20, 2022

fix(pf4-style): expose scss variables (#110)

a62c2be

cdcabrera added a commit that referenced this pull request Sep 20, 2022

feat(modal): discovery-148 pf4 modal wrapper (#110)

8bc5c28

* locales, modal default string * modal, consistent pf4 wrapper, allow backdrop disable

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(modal): discovery-148 pf4 modal wrapper #110

feat(modal): discovery-148 pf4 modal wrapper #110

cdcabrera commented Jun 30, 2022 •

edited

Loading

codecov-commenter commented Jun 30, 2022

feat(modal): discovery-148 pf4 modal wrapper #110

feat(modal): discovery-148 pf4 modal wrapper #110

Conversation

cdcabrera commented Jun 30, 2022 • edited Loading

What's included

Chores

Bug Fixes

Features

Notes

How to test

Coverage and basic unit test check

Check the build

Example

Updates issue/story

codecov-commenter commented Jun 30, 2022

Codecov Report

cdcabrera commented Jun 30, 2022 •

edited

Loading