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

Miscelaneous updates #132

Merged
merged 6 commits into from
Feb 12, 2025
Merged

Miscelaneous updates #132

merged 6 commits into from
Feb 12, 2025

Conversation

yoannmoinet
Copy link
Member

@yoannmoinet yoannmoinet commented Feb 11, 2025

What and why?

In order to ease the reviews of #125 and #119 I'm splitting out some small updates.

How?

  • Add auth to the doRequest helper.
  • Add more data into the logger.
  • Fix injection plugin to avoid leaking into global context (conflicting variable names).
  • Update the bundling to allow plugins to add their own files to build through the toBuild entry in the package.json (useful for the Browser SDK injection, and later, the React integration).

@yoannmoinet yoannmoinet changed the base branch from master to yoann/v2.5.0 February 11, 2025 09:09
@yoannmoinet yoannmoinet marked this pull request as ready for review February 11, 2025 09:12
@yoannmoinet yoannmoinet requested a review from a team as a code owner February 11, 2025 09:12
@yoannmoinet yoannmoinet requested review from sarenji, elbywan and Ayc0 and removed request for a team and sarenji February 11, 2025 09:12
@yoannmoinet yoannmoinet mentioned this pull request Feb 12, 2025
@yoannmoinet yoannmoinet merged commit d30f46f into yoann/v2.5.0 Feb 12, 2025
3 checks passed
@yoannmoinet yoannmoinet deleted the yoann/miscelaneous-updates branch February 12, 2025 08:55
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