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

Fix newArch issue with no feedback on android #865

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

MrRefactor
Copy link
Collaborator

@MrRefactor MrRefactor commented Aug 22, 2024

Summary

Test Plan

Just trust me ok?

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
iOS ✅❌
Android ✅❌

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)

Copy link
Member

@troZee troZee left a comment

Choose a reason for hiding this comment

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

@MrRefactor this should go as a breaking change, right?

@WoLewicki
Copy link
Contributor

Did you manage to dig down into how yoga resolves those views? I'd be curious to follow the progress on this issue 👀

@MrRefactor
Copy link
Collaborator Author

Did you manage to dig down into how yoga resolves those views? I'd be curious to follow the progress on this issue 👀

still looking into it, trying to figure out other solution

@MrRefactor MrRefactor merged commit 32df06a into master Aug 28, 2024
2 checks passed
@MrRefactor MrRefactor deleted the fix-tmp/newArch-android-feedback branch August 28, 2024 11:57
github-merge-queue bot referenced this pull request in valora-inc/wallet Sep 7, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[react-native-pager-view](https://github.com/callstack/react-native-pager-view)
| [`^6.3.4` ->
`^6.4.1`](https://renovatebot.com/diffs/npm/react-native-pager-view/6.3.4/6.4.1)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/react-native-pager-view/6.4.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/react-native-pager-view/6.4.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/react-native-pager-view/6.3.4/6.4.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/react-native-pager-view/6.3.4/6.4.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>callstack/react-native-pager-view
(react-native-pager-view)</summary>

###
[`v6.4.1`](https://github.com/callstack/react-native-pager-view/releases/tag/v6.4.1)

[Compare
Source](https://github.com/callstack/react-native-pager-view/compare/v6.4.0...v6.4.1)

#### What's Changed

- feat: add load method by
[@&#8203;WoLewicki](https://github.com/WoLewicki) in
[https://github.com/callstack/react-native-pager-view/pull/867](https://github.com/callstack/react-native-pager-view/pull/867)
- Update README.md by
[@&#8203;yilinjuang](https://github.com/yilinjuang) in
[https://github.com/callstack/react-native-pager-view/pull/870](https://github.com/callstack/react-native-pager-view/pull/870)
- fix(android): newArch issue with no feedback on android by
[@&#8203;MrRefactor](https://github.com/MrRefactor) in
[https://github.com/callstack/react-native-pager-view/pull/865](https://github.com/callstack/react-native-pager-view/pull/865)

#### New Contributors

- [@&#8203;WoLewicki](https://github.com/WoLewicki) made their
first contribution in
[https://github.com/callstack/react-native-pager-view/pull/867](https://github.com/callstack/react-native-pager-view/pull/867)
- [@&#8203;yilinjuang](https://github.com/yilinjuang) made
their first contribution in
[https://github.com/callstack/react-native-pager-view/pull/870](https://github.com/callstack/react-native-pager-view/pull/870)

**Full Changelog**:
callstack/react-native-pager-view@v6.4.0...v6.4.1

###
[`v6.4.0`](https://github.com/callstack/react-native-pager-view/releases/tag/v6.4.0)

[Compare
Source](https://github.com/callstack/react-native-pager-view/compare/v6.3.4...v6.4.0)

Changelog:

- fix(android): Fix android newArch build
([#&#8203;864](https://github.com/callstack/react-native-pager-view/issues/864))
([`12e29d5`](https://github.com/callstack/react-native-pager-view/commit/12e29d5))
[@&#8203;MrRefactor](https://github.com/MrRefactor)
- fix(ios): Change RNCViewPagerComponenDescriptor import
([#&#8203;863](https://github.com/callstack/react-native-pager-view/issues/863))
([`0f4bac8`](https://github.com/callstack/react-native-pager-view/commit/0f4bac8))
[@&#8203;MrRefactor](https://github.com/MrRefactor)
- fix(ios): visionOS builds
([`cca654a`](https://github.com/callstack/react-native-pager-view/commit/cca654a))
[@&#8203;okwasniewski](https://github.com/okwasniewski)
- chore(example): migrate example to `react-native-test-app`
([`ea11c94`](https://github.com/callstack/react-native-pager-view/commit/ea11c94))
[@&#8203;szymonrybczak](https://github.com/szymonrybczak)
- fix(android): child argument types in addView methods
([#&#8203;857](https://github.com/callstack/react-native-pager-view/issues/857))
([`993928c`](https://github.com/callstack/react-native-pager-view/commit/993928c))
[@&#8203;szydlovsky](https://github.com/szydlovsky)
- fix(js): remove RCTEventEmitter workaround
([#&#8203;858](https://github.com/callstack/react-native-pager-view/issues/858))
([`1e61a59`](https://github.com/callstack/react-native-pager-view/commit/1e61a59))
[@&#8203;MrRefactor](https://github.com/MrRefactor)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 5pm,every weekend" in timezone
America/Los_Angeles, Automerge - "after 5pm,every weekend" in timezone
America/Los_Angeles.

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/valora-inc/wallet).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguNTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsibnBtIiwicmVub3ZhdGUiXX0=-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: valora-bot <[email protected]>
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.

3 participants