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

Bump react-aria from 3.34.1 to 3.34.3 #5038

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 16, 2024

Bumps react-aria from 3.34.1 to 3.34.3.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Sep 16, 2024
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/react-aria-3.34.3 branch 2 times, most recently from b765af6 to 94a1813 Compare September 16, 2024 15:17
@jwhitlock jwhitlock added the 🛑 Do Not Merge Do not merge this PR, even if approved. label Sep 16, 2024
@jwhitlock
Copy link
Member

jwhitlock commented Sep 16, 2024

Confirmed in the netlify preview that this update continues to create two random addresses when one is requested. Tracked in MPP-3893.

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/react-aria-3.34.3 branch from 94a1813 to f04a8b6 Compare September 16, 2024 21:57
Copy link
Contributor Author

dependabot bot commented on behalf of github Sep 16, 2024

Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update by commenting @dependabot rebase.

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/react-aria-3.34.3 branch 2 times, most recently from 6a484bc to d4a0795 Compare September 23, 2024 15:47
Bumps [react-aria](https://github.com/adobe/react-spectrum) from 3.34.1 to 3.34.3.
- [Release notes](https://github.com/adobe/react-spectrum/releases)
- [Commits](https://github.com/adobe/react-spectrum/compare/react-aria@3.34.1...react-aria@3.34.3)

---
updated-dependencies:
- dependency-name: react-aria
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/react-aria-3.34.3 branch from d4a0795 to 92307a8 Compare September 23, 2024 16:00
@groovecoder groovecoder requested a review from Vinnl September 23, 2024 17:19
@groovecoder groovecoder self-assigned this Sep 23, 2024
@groovecoder groovecoder removed the 🛑 Do Not Merge Do not merge this PR, even if approved. label Sep 23, 2024
Copy link
Collaborator

@Vinnl Vinnl left a comment

Choose a reason for hiding this comment

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

Change looks good to me. I also noticed that it's no longer possible to reach the menu items via the down arrow (or other keys) on the keyboard, but that was already the case in main. That's an accessibility issue, but I'll leave the prioritisation to the team :)

@@ -372,7 +372,6 @@ const AliasTypeMenuItem = (props: AliasTypeMenuItemProps) => {
{
key: props.item.key,
isDisabled: props.item.isDisabled,
onAction: props.onAction,
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you should also be able to remove line 365 now.

Copy link
Member

Choose a reason for hiding this comment

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

Our AliasGenerationButton component passed onAction to both useMenu and
useMenuItem hooks. react-aria has deprecated onAction handlers from the
menu item level in favor of handling actions at the menu level. Previous
versions of react-aria reconciled multiple handlers, but 3.43.3 does
not. So our handler was being called twice, causing MPP-3893.

(Note: the handler for custom masks was being called twice, but since it
did not POST to create an address, it did not create 2 masks.)
@groovecoder groovecoder force-pushed the dependabot/npm_and_yarn/react-aria-3.34.3 branch from c574271 to 00109ca Compare September 26, 2024 14:05
@groovecoder groovecoder added this pull request to the merge queue Sep 26, 2024
Merged via the queue into main with commit 4cde843 Sep 26, 2024
29 checks passed
@groovecoder groovecoder deleted the dependabot/npm_and_yarn/react-aria-3.34.3 branch September 26, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants