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

Chore/minimise bundle size #884

Open
wants to merge 6 commits into
base: staging
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
"prettier"
],
"rules": {
"@typescript-eslint/no-unused-vars": "off",
"@typescript-eslint/no-explicit-any": "off",
"@typescript-eslint/ban-ts-comment": "off",
"@typescript-eslint/ban-types": "off",
"@typescript-eslint/indent": "off",
Expand Down
23 changes: 8 additions & 15 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
> Description

Feature release including multi-hook support, improved UX for off chain token holders and general improvements.
Performance and code quality improvements.

### Upgrade Steps

Expand All @@ -12,26 +12,19 @@ Feature release including multi-hook support, improved UX for off chain token ho

### New Features

- Extended the configuration options for 'active' mode ui text
- Off Chain Token Issuer UX improvements to notify the end user when the page will re-direct
- Re-direct view state UX improvements for 'active' mode
- Re-direct accept / deny options given to end user (to cancel the loading of tokens)
- Multi-Hook support added
- Added selected token issuer keys to 'tokens-selected' event hook
- Explicitly include ethers library availabilty via Token Negotiator library interface `client.externalUtils.evm.ethers`
- Aligned on chain authentication with off chain user interface (for single token authentication on the client side). Multi token on chain authentication is not yet supported (via the current library features).
[none]

### Bug Fixes

- TS interface custom view
- Single token off chain authentication incompatibility interface fix
- EAS authentication fixed via attestation.id updated dependency @tokenscript/attestation to version "0.7.0-rc.2"
[none]

### Performance / Quality Improvements

- Reduced the duplication of variable definitions (moved to a common constants file).
- Increased unit test coverage
- Reduced cognitive complexity of functions
- Removed deprecated TODO's or those logged for future changes in product roadmap
- Upgraded package dependencies
- Additional Code splitting added to reduce the package size

**Full Change log**:

https://github.com/TokenScript/token-negotiator/compare/v3.1.1....v3.2.0
https://github.com/TokenScript/token-negotiator/compare/v3.2.0....v3.2.1
Loading