Skip to content

Conversation

@artyorsh
Copy link
Contributor

@artyorsh artyorsh commented May 22, 2025

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

#868

Describe the solution you've provided

Provides Requestor configuration in correct order.

Describe alternatives you've considered

None.

Additional context

Fixes #868

Edit: in functions with many arguments, consider arguments with an object, so that it's more clear what values are assigned to which keys.

@artyorsh artyorsh requested a review from a team as a code owner May 22, 2025 15:49
@kinyoklion
Copy link
Member

@artyorsh Thank you for finding this! I will do some validation and then get it released.

@artyorsh
Copy link
Contributor Author

Ouch :) Why it fails?

I unfortunately don't have more time for the issue, hope it is incorrectly implemented test and there is enough context already.

@kinyoklion
Copy link
Member

Ouch :) Why it fails?

I unfortunately don't have more time for the issue, hope it is incorrectly implemented test and there is enough context already.

Some of the builds require permissions that are not compatible with forks. We probably should disable those steps, but I will check the builds to make sure everything is fine.

@kinyoklion kinyoklion merged commit 2e095c5 into launchdarkly:main May 22, 2025
24 of 25 checks passed
@github-actions github-actions bot mentioned this pull request May 22, 2025
kinyoklion pushed a commit that referenced this pull request May 22, 2025
🤖 I have created a release *beep* *boop*
---


<details><summary>jest: 0.1.7</summary>

##
[0.1.7](jest-v0.1.6...jest-v0.1.7)
(2025-05-22)


### Dependencies

* The following workspace dependencies were updated
  * dependencies
* @launchdarkly/react-native-client-sdk bumped from ~10.10.1 to ~10.10.2
</details>

<details><summary>react-native-client-sdk: 10.10.2</summary>

##
[10.10.2](react-native-client-sdk-v10.10.1...react-native-client-sdk-v10.10.2)
(2025-05-22)


### Bug Fixes

* Requestor initialized with wrong withReasons and useReport config
([#869](#869))
([2e095c5](2e095c5))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

[React Native] Inconsistent withReasons + initialConnectionMode behavior

2 participants