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.

Sign up for GitHub

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

Jump to bottom

[Release] 3.2.2 #382

Merged
merged 17 commits into from
Feb 28, 2024
Merged

[Release] 3.2.2 #382

merged 17 commits into from
Feb 28, 2024

Conversation

Ansonhkg
Copy link
Collaborator

@Ansonhkg Ansonhkg commented Feb 22, 2024 •
edited
Loading

Version: 3.2.2

Pull Requests

Checklist:

  • Testing: All new features and bug fixes must have passed unit and e2e tests. Automated test results must be reviewed and approved by at least one member of the JS-SDK team.
  • Documentation: Any changes to the JS-SDK, including new features, modifications, or bug fixes, must be accompanied by updated documentation. This includes API docs, changelogs, and migration guides for breaking changes
  • Deprecation Notices: If the release includes deprecating existing features, a clear deprecation notice (adding deprecated tag to related function) must be provided, including the timeline for when the feature will be removed and guidance for alternative solutions.
  • No tsc warnings: If your branch much follow the configured tsc rules meaning no new warnings or errors should introduced in your branch.
  • Release notes: Contact the docs team to prepare the release note, highlighting new features, improvements, bug fixes, and known issues.

Ansonhkg and others added 15 commits February 22, 2024 13:14
@Ansonhkg
init patch
271b033
@Ansonhkg
Feature/lit 2570 js sdk fix build (#387)
1628810 
* feat: make capacityTokenId optional

* chore: add test

* fix: Conditionally include nft_id

* fix: replace nx executor to @nrwl/js:tsc

* fix: replace nx executor

* feat: migrated nx to 15.9.0

* feat: migration nx from 15.9.0 -> 16.10.0

* feat: migrate nx to 17.3.0

* fix: yarn:test:unit command

* chore: log cc nft token info

* fix: undefined typings in package.json
@glitch003 @Ansonhkg
remove old v0 errors from js sdk. provide error details to the user (#…
5464279 
…385)

* remove old v0 errors from js sdk.  provide error details to the user

* building now seems to consistently work

* refactored error handling to make it cleaner

* Remove unused function

* okay also added requestId in a few places

* ran the linter

* fix: replace nx executor

* feat: add gitignore

---------

Co-authored-by: Ansonhkg <[email protected]>
@Ansonhkg
feat: cache enabled
6e6a12e
@Ansonhkg
chore: update .gitignore
747c4af
@Ansonhkg
Merge remote-tracking branch 'origin/staging/3.2.2' into staging/3.2.2
90ea7df
@Ansonhkg
chore: enable parallel
d2f13cb
@Ansonhkg
feat: make capacityTokenId optional (#384)
a8ba384 
* feat: make capacityTokenId optional

* chore: add test

* fix: Conditionally include nft_id
@Ansonhkg
Update yarn.lock to match package.json changes
ea0e887
@Ansonhkg
fix: ReferenceError: crypto is not defined
a7b9d65
@Ansonhkg
fix: ignore nx cache directory on linting
104aa6d
@Ansonhkg
fix: ignore tools directory on linting
3bd87a6
@Ansonhkg
fix: linting
318d05f
@Ansonhkg
fix: '
1808b7a 
ReferenceError: require is not defined in ES module scope, you can use import instead'
@Ansonhkg
fix: lint
ac9abb6
@Ansonhkg Ansonhkg marked this pull request as ready for review February 28, 2024 03:27
@Ansonhkg Ansonhkg merged commit 987107c into master Feb 28, 2024
4 checks passed
@Ansonhkg Ansonhkg deleted the staging/3.2.2 branch February 28, 2024 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joshLong145 joshLong145 Awaiting requested review from joshLong145

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ansonhkg @glitch003