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

Updates for Near Org #10

Merged
merged 4 commits into from
Aug 9, 2024
Merged

Updates for Near Org #10

merged 4 commits into from
Aug 9, 2024

Conversation

calebjacob
Copy link
Collaborator

Related To: near/nearorg#68

  • Fixed lodash import to support proper tree shaking to resolve unintended package bloat. This reduces our package size by roughly 500KB!
  • Added rollup plugin to visualize package output size based on source code and dependencies.
  • Adding and modifying various components to support Near Org refactor.

@calebjacob calebjacob added the enhancement New feature or request label Aug 9, 2024
@calebjacob
Copy link
Collaborator Author

Hey @shelegdmitriy I'm going to go ahead and merge this to get a head start on the nearorg PR and get these changes published. Let me know if you see any issues. Thanks!

@calebjacob calebjacob merged commit 83ca810 into main Aug 9, 2024
2 checks passed
@calebjacob calebjacob deleted the feat/near-org-updates branch August 9, 2024 18:35
Comment on lines +5 to +6
width: 64px;
height: 64px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to control icon size, but we can improve this later

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great point! A lot of these components can continue to improve. Feel free to open up tickets and push PR's for any improvements you'd like to see :)

Copy link

@shelegdmitriy shelegdmitriy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants