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

v7 Milestone #481

Merged
merged 45 commits into from
Jul 6, 2022
Merged

v7 Milestone #481

merged 45 commits into from
Jul 6, 2022

Conversation

chawes13
Copy link
Contributor

@chawes13 chawes13 commented Aug 31, 2021

Items Addressed

Features/Bugs

  1. Resolves Change hideLabel prop name for clarity #560
  2. Resolves Give Select inputs a placeholder value by default #233
  3. Resolves Set aria-disabled instead of disabled on Button / SubmitButton #325
  4. Defaults Radio and Checkbox group labels to legends
  5. Resolves Stop using style prop for buttons #293
  6. Resolves FileInput doesn't support multiple files #346
  7. Resolves Cloudinary file input will not accept the same file after a failed upload #381
  8. Resolves File Input does not get reset after redux-form reset() #387
  9. Resolves Use className for spinner instead of id #411
  10. Resolves Make options and value required in TabBar #419
  11. Resolves Add remove button to default file input previews #347
  12. Resolves Remove SetterLink component #524
  13. Resolves Allow Modal's onClose to be null and derive other props from it #517
  14. Resolves FlashMessage onDismiss should receive message object (including id) #515
  15. Resolves Button not accepting a ref #536
  16. Resolves Ability pass other props or rest to Cell Component #458
  17. Resolves Don't overwrite modal-inner when classname is passed #518
  18. Resolves Allow for caption to be specified for Table components #463
  19. Resolves MaskedInput- props not getting passed in #537
  20. Resolves Add issue templates #561
  21. Resolves Apply custom class name to wrapper when input is disabled #530
  22. Resolves Defaulting to the body element breaks a11y on Firefox #496
  23. Resolves Upgrade DateInput to use ^4.0.0 of react-datepicker #305
  24. Resolves Remove dependency on lp-hoc #414
  25. Resolves Checkbox/Radio group input prop types should accept bool #532
  26. Resolves Add support for React 17 #566
  27. Resolves Apply class name only to CheckboxGroup/RadioGroup container #467

Critical Security Issues

  1. https://github.com/LaunchPadLab/lp-components/security/dependabot/84
  2. https://github.com/LaunchPadLab/lp-components/security/dependabot/62

Contributors

  • Rachel K
  • David P
  • Conor H
  • George L
  • Angel M
  • Nikki D
  • Alex J
  • Ji P
  • Ifat R
  • Mike W
  • Alejandro M
  • Danny P

Author Checklist

  • Add unit test(s)
  • Update version in package.json (see the versioning guidelines)
  • Update documentation (if necessary)
  • Add story to storybook (if necessary)
  • Assign dev reviewer

Rachel Killackey and others added 17 commits April 17, 2018 13:52
* add hideLabel prop, story

* update showLabel logic, add new story

* remove import

* Version bump: 3.7.0

* add input label test, fix range input test

* lint

* change range label prop name
* add default placeholder

* Version bump: 3.12.0
* Default label component to a legend for checkbox group

* Default label component to a legend for radio group
# Conflicts:
#	docs.md
#	package.json
#	src/forms/buttons/button.js
#	test/forms/inputs/select.test.js
* Remove custom style prop on button

* Remove disabled button test

* Cleanup

* Some formatting

* Priority -> variant
* Initial refactor

* Replace remove button with customizable component that accepts handler

* POC for workaround with files failing to load (mostly a upstream cloudinary issue)

* Refactor to a controlled file input (#388)

* Initial proof-of-concept

* Clean up logic and add dev hints via prop types

* Always return an array

* Read files synchronously to avoid overwrites

* Modify markup to allow for focus styling

* Code review feedback

* Make cloudinary accept a file input and set the response appropriately

* Extract helpers into shared folder

* Override read files utility and clean up based on code review

* filter invalid props from getting passed to error label (incl. label = false)

* Return files on cloudinary read

* Code clean-up

* Update CloudinaryFileInput docs

* Update FileInput docs

* Bump size limit

* Bump to milestone

* Fix failing specs

* Remove duplicate import

* Fix merge conflicts

* Add specs for multiple file input

* Add coverage for inverse scenario

* Add missing tests for remove functionality

* Refactor FileInput using hooks

* Use hook import directly

* Safely mock global object in an isolated way

* Wrap component in act to ensure state updates have settled

* Add tests for error capture

* Cover ternary branching conditions

* Update migration guide

* Code review feedback

* Add new test

* Manually updating docs

* Refactor to always store values in an array

* Update migration guide based on input value change

* Update prop types

* Do not change initial value if not an array (it will flag form as dirty)

* Add thumbnail story

* Code review feedback

* Globally replace redux-forms

* Reference array instead of single file
ghrlee and others added 6 commits January 11, 2022 15:57
* change id to classname on spinner component

* revert inadvertnet commit

* add className to spinner component

* passing tests and update version

* add notes on the migration guide file

* adding feedback changes

* fix migration guide

* update language in migration guide

* additional feedback comments

* update default prop

* add destructuring to props

* Small tweaks

* Add missing period

Co-authored-by: Conor Hawes <[email protected]>
* Made options & value as required for the TabBar component

* bump version

* update docs, storybook, tests. remove unneeded activeValue logic

* Small documentation tweaks

* Update docs and travis

* Address linting errors

* Address unrelated lint error

Co-authored-by: Conor <[email protected]>
* show clear button for single file upload

* formatting

* tests for clear file input btn

* make remove button for single file consistent with remove button for multiple files

* update docs

* run yarn docs manually

* Add small note to migration guide

Co-authored-by: Conor Hawes <[email protected]>
* Remove SetterLink

* Update docs
* Prevent all methods of closure

* Update migration guide

* Make isOpen prop less confusing

* Manually update docs
@chawes13 chawes13 changed the title V6 Milestone V7 Milestone Jun 14, 2022
chawes13 and others added 6 commits June 14, 2022 13:39
* Hotfix

* Be more specific

* Fix select test warnings

* Fix textarea test warnings

* Fix file input test warnings

* Dedupe yarn.lock file to help resolve issues
#503)

* allow cell to take custom properties like aria and data-cy per cell by column

* versioning to 5.4.3

* add filterInvalidProps to DefaultCellComponent render

* add back space

* add sortable-table.test.js test for addition;

* remove custom class for valid dom prop

* add test for invalid DOM props on column

* Tweak formatting

* Remove unnecessary attributes

Co-authored-by: Conor Hawes <[email protected]>
@chawes13 chawes13 linked an issue Jun 15, 2022 that may be closed by this pull request
* Add functionality

* Add specs

* Add story

* Update migration guide

* Tweak migration guide
@chawes13 chawes13 linked an issue Jun 16, 2022 that may be closed by this pull request
* render caption if passed in as a prop

* remove semicolons i missed

* migration guide

* Render caption as table's first descendant

* Add story

* Fix typo

* Rename file to match convention

* Fix more typos

* Add caption tests

Co-authored-by: Conor Hawes <[email protected]>
* Add issue templates

* Update FEATURE_REQUEST.md
* Add support for additional undocumented props

* Update docs and support callback ref as a prop

* Actually update docs
* Add disabled class at top level

* Update spec
* Switch selector

* Apply aria-hidden on the browser

* Update documentation

* Fix prop documentation 😳
* Add missing test case

* Upgrade to v2

* Upgrade to v3

* Update to v4

* Tweak formatting

* Update migration guide

* Update docs

* Use calendar ref

* Simplify implementation

* Generate docs after merge

* Write a better comment
* Do not use input prop types for a component that is not an input

* Isolate components to allow for checkbox and radio groups to receive bools

* Reuse type

* Fix tabbar prop type issue

* Remove unused import

* Update docs
* Remove unused sortable imports

* Remove toggle usage from most components

* Update input-label

* Remove unused hoc

* Remove toggle and outside click hocs

* Port over most files from lp-hoc's cloudinary uploader

* Bring over cloudinary uploader and tweak tests

* Clean up util usage

* Use cloudinary hoc mock in file input test

* Officially remove lp-hoc from library

* Suppress unnecessary errors shown in modal

* Lint

* Add back export for cloudinary uploader util

* Update migration guide

* Fix heading formatting

* Update docs

* Always set a boolean value

* Add docs about breaking change

* Update docs after latest merge

* Be better at writing tests

* Coerce using Boolean constructor
* Fix watch infinite loop

* Avoid passing down class name and allow localized input prop overrides

* Update docs

* Add stories

* Add examples
@chawes13 chawes13 marked this pull request as ready for review July 4, 2022 20:26
@chawes13 chawes13 changed the title V7 Milestone v7 Milestone Jul 6, 2022
@chawes13 chawes13 merged commit 3b6578b into main Jul 6, 2022
@chawes13 chawes13 deleted the v6 branch July 6, 2022 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment