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

chore(file-uploader): Tagged release for testing #3094

Merged
merged 111 commits into from
Nov 28, 2022

Conversation

ErikCH
Copy link
Contributor

@ErikCH ErikCH commented Nov 28, 2022

Description of changes

Tagged release for File uploader

Issue #, if available

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Tests are updated
  • No side effects or sideEffects field updated
  • Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Erik Hanchett and others added 30 commits November 14, 2022 14:40
feat(file-uploader): Added new overrides for Previewer and Tracker
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
feat(file-uploader): Prop for Auto Proceed
feat(file-uploader): Prop to turn off or on images in Tracker
* refactor(docs): generate props table data

* fix(docs): props table remove sanitizer

* fix(docs): props table add code highlight

* chore(docs): convert Markdown Description to HTML

* chore(docs): props table sort props order, refactor PropsTableTab

* chore(docs): props table heading link and required badge

* refactor(docs): delete props-table.mdx and use PropsTableTab component

* chore(docs): props table optimize required badge

* chore(docs): props table update table data

* chore(docs): props table refactor data to use map so that the order is more stable

* test(docs): add props table data snapshot test

* chore(docs): update README

* Update docs/src/components/propsTable/PropsTableBody.tsx

Co-authored-by: Heather Buchel <hbuchel@gmail.com>

* Update docs/src/components/propsTable/PropsTableTab.tsx

Co-authored-by: Heather Buchel <hbuchel@gmail.com>

* feat(docs): props table css copy from PR2435

* chore(docs): remove props-table-data.json

* chore(docs): gitignore props-table.json

* chore(docs): props table mdnUrl

* chore(docs): props table replace p with View

* refactor(docs): use react-markdown

* refactor(docs): use Text to replace View as p

* chore(docs): props table add placeholder

* fix: props table expander focus style (#3006)

Co-authored-by: Heather Buchel <buchel@amazon.com>

Co-authored-by: 0618 <o.6180339@gmail.com>
Co-authored-by: Heather Buchel <hbuchel@gmail.com>
Co-authored-by: Heather Buchel <buchel@amazon.com>
* refactor(docs): remove numberOfPages from broken link test

* fix(docs): broken link test refactor hrefWorks

* Revert "fix(docs): broken link test refactor hrefWorks"

This reverts commit 120c0fe.

* chore(docs): add comments for broken link test

* feat(docs): link checker add param

* chore(docs): cypress test disable video

* refactor(docs): link checker cy.get internal and external links

Co-authored-by: 0618 <o.6180339@gmail.com>
Bumps [deep-object-diff](https://github.com/mattphillips/deep-object-diff) from 1.1.7 to 1.1.9.
- [Release notes](https://github.com/mattphillips/deep-object-diff/releases)
- [Commits](https://github.com/mattphillips/deep-object-diff/commits)

---
updated-dependencies:
- dependency-name: deep-object-diff
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* subscribe now exposes facade values

* Add changeset

* Update return type

* Unit test!

* change test name

* perf: store facade values directly

* Update unit test

* Minor optimization

* Inline mock service

* Don't return unsubscribe directly

* Update .changeset/smooth-moons-occur.md

* Update .changeset/smooth-moons-occur.md

* Fix unit test
chore(file-uploader): Removed fileNames prop and added warning for users for required props
Erik Hanchett and others added 5 commits November 25, 2022 10:40
@ErikCH ErikCH requested a review from a team as a code owner November 28, 2022 18:04
@changeset-bot
Copy link

changeset-bot bot commented Nov 28, 2022

🦋 Changeset detected

Latest commit: fa7db24

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@aws-amplify/ui Minor
@aws-amplify/ui-react Minor
@aws-amplify/ui-react-core Patch
@aws-amplify/ui-react-native Patch
@aws-amplify/ui-vue Patch
@aws-amplify/ui-angular Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@lgtm-com
Copy link

lgtm-com bot commented Nov 28, 2022

This pull request fixes 3 alerts when merging df25024 into 41bde68 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class
  • 1 for Useless conditional

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 28, 2022

This pull request fixes 3 alerts when merging a8ad125 into 41bde68 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class
  • 1 for Useless conditional

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 28, 2022

This pull request fixes 3 alerts when merging 032f403 into 41bde68 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class
  • 1 for Useless conditional

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@ErikCH ErikCH temporarily deployed to ci November 28, 2022 21:00 Inactive
@ErikCH ErikCH temporarily deployed to ci November 28, 2022 21:00 Inactive
@ErikCH ErikCH temporarily deployed to ci November 28, 2022 21:00 Inactive
@ErikCH ErikCH temporarily deployed to ci November 28, 2022 21:00 Inactive
@lgtm-com
Copy link

lgtm-com bot commented Nov 28, 2022

This pull request fixes 3 alerts when merging fa7db24 into 41bde68 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class
  • 1 for Useless conditional

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@dbanksdesign dbanksdesign temporarily deployed to ci November 28, 2022 22:00 Inactive
@dbanksdesign dbanksdesign temporarily deployed to ci November 28, 2022 22:00 Inactive
@dbanksdesign dbanksdesign temporarily deployed to ci November 28, 2022 22:00 Inactive
@dbanksdesign dbanksdesign temporarily deployed to ci November 28, 2022 22:00 Inactive
@ErikCH ErikCH merged commit bf627b4 into fileuploader/release Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants