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-native-haptic-feedback from 2.2.0 to 2.3.3 in /VAMobile #9873

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 14, 2024

Bumps react-native-haptic-feedback from 2.2.0 to 2.3.3.

Release notes

Sourced from react-native-haptic-feedback's releases.

v2.3.3

Missing export of enum value.

#139

v2.3.2

Fix iOS new arch issues and typescript types

Thanks to

Fix pod install & implement testing environment

Fix of union types Thanks to #129

Also added testing environments correctly to prevent future broken builds

React-Native Compatibility > 0.71

Aligning the requirements for new arch

Thanks to @​AlirezaHadjar #117 @​buschco #121 @​andhac #123 @​WoLewicki #125

Commits
  • 8cdf0f3 remove yarn.lock
  • 7668b09 adjust example project usage
  • 5bb92f6 Merge pull request #139 from huextrat/fix/missing-types-export
  • 6c5b540 fix: export types from index
  • 3254566 version bump
  • d1516d5 Merge pull request #135 from huextrat/fix/types
  • 65c8b10 Merge pull request #136 from huextrat/build/bump-compile-target-sdk
  • ae7a3f8 build(android): bump compile & target SDK to 34
  • 16508f2 fix: types
  • 55b7150 Merge pull request #133 from WoLewicki/@​wolewicki/fix-newarch
  • Additional commits viewable in compare view

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)

Bumps [react-native-haptic-feedback](https://github.com/mkuczera/react-native-haptic-feedback) from 2.2.0 to 2.3.3.
- [Release notes](https://github.com/mkuczera/react-native-haptic-feedback/releases)
- [Commits](mkuczera/react-native-haptic-feedback@v2.2.0...v2.3.3)

---
updated-dependencies:
- dependency-name: react-native-haptic-feedback
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot requested review from a team as code owners October 14, 2024 13:00
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Oct 14, 2024
@dependabot dependabot bot requested a review from timwright12 October 14, 2024 13:00
@dependabot dependabot bot added the javascript Pull requests that update Javascript code label Oct 14, 2024
@Sparowhawk
Copy link
Contributor

@theodur I updated the pods and I am still getting the error if you can take a look

@theodur
Copy link
Contributor

theodur commented Oct 31, 2024

@Sparowhawk The issue was the package needed to be mocked in our unit test setup. They must have changed something with how the package is referenced in the latest release. I'm running Detox tests to make sure everything passes before approving and merging.

Edit: This is one of those packages that, even though should work with a minor update, should still be tested physically, since there's no way to be certain that haptics still works from the tests. I kicked on an on-demand build.

Copy link
Contributor

@theodur theodur left a comment

Choose a reason for hiding this comment

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

Detox tests passed, and I confirmed haptics is still working on a local build

@github-actions github-actions bot added the FE-With QA A PR waiting for QA Activity label Oct 31, 2024
@theodur theodur merged commit c5d48d5 into develop Oct 31, 2024
99 checks passed
@theodur theodur deleted the dependabot/npm_and_yarn/VAMobile/react-native-haptic-feedback-2.3.3 branch October 31, 2024 19:50
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 FE-With QA A PR waiting for QA Activity javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants