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: Hardcode webclient version #381

Merged
merged 5 commits into from
Mar 7, 2023
Merged

Conversation

limhjgrace
Copy link
Contributor

Applications using cjs run into a TypeError due to import issues involving the version.json file. Related issue: #371

As a temporary fix, we will hardcode the webclient version.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@limhjgrace limhjgrace requested a review from qhanam March 7, 2023 18:42
@limhjgrace limhjgrace requested a review from ps863 March 7, 2023 18:47
@ps863
Copy link
Member

ps863 commented Mar 7, 2023

question: does "resolveJsonModule": true remain true if we are hardcoding the web client version?

@limhjgrace
Copy link
Contributor Author

question: does "resolveJsonModule": true remain true if we are hardcoding the web client version?

has been removed

@qhanam
Copy link
Member

qhanam commented Mar 7, 2023

Related to #381

@limhjgrace limhjgrace merged commit bc1c15f into aws-observability:main Mar 7, 2023
limhjgrace added a commit to limhjgrace/aws-rum-web that referenced this pull request Mar 7, 2023
* fix: Hardcode webclient version

* fix: Update test variable value

* fix: update webclient version

* fix: revert tsconfig changes related to webclient versioning feature

* fix: remove version updating
limhjgrace added a commit to limhjgrace/aws-rum-web that referenced this pull request Mar 7, 2023
* fix: Hardcode webclient version

* fix: Update test variable value

* fix: update webclient version

* fix: revert tsconfig changes related to webclient versioning feature

* fix: remove version updating
limhjgrace added a commit that referenced this pull request Mar 7, 2023
* fix: Hardcode webclient version

* fix: Update test variable value

* fix: update webclient version

* fix: revert tsconfig changes related to webclient versioning feature

* fix: remove version updating
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.

None yet

3 participants