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

Release 210.0.0 #4742

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Release 210.0.0 #4742

merged 1 commit into from
Sep 30, 2024

Conversation

adonesky1
Copy link
Contributor

Explanation

This is a RC for v210.0.0. See changelog for more details

[5.1.0]

Changed

  • Batch processing now considers both origin and networkClientId, ensuring requests targeting different networks are processed separately. (#4718)
  • Incoming requests to enqueueRequest now must include a networkClientId; an error is thrown if it's missing. This was previously a required part of the type but since consumers like the extension do not have extensive typescript coverage this wasn't definitively enforced. (#4718)

@adonesky1 adonesky1 requested a review from a team as a code owner September 27, 2024 17:31
@adonesky1 adonesky1 requested review from Gudahtt and a team September 27, 2024 17:45
@jiexi
Copy link
Contributor

jiexi commented Sep 27, 2024

As discussed in our call, I think 6.0.0 would be good because of the breaking change with the request.networkClientId enforcement, but it's not really a big deal. 5.1.0 is fine

@adonesky1 adonesky1 merged commit 8d61ac9 into main Sep 30, 2024
116 checks passed
@adonesky1 adonesky1 deleted the release/210.0.0 branch September 30, 2024 14:13
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.

2 participants