-
Notifications
You must be signed in to change notification settings - Fork 16
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: remove unused exports #700
Conversation
✅ Deploy Preview for testitori ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for teritori-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Could you rename |
Here
FeedPostShareModalProps as ShareButtonProps (Or remove this interface, cause there is only one property), since this file is removed: https://github.com/TERITORI/teritori-dapp/blob/ce0d7ea427b993e0dc4f1c8a8006bb821933d445/packages/components/socialFeed/SocialActions/FeedPostShareModal.tsx
|
Please fix lint-and-build and update the branch with origin/main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got this warning : Attempted import error: 'GradientType' is not exported from './GradientText'
.
(I don't know why, it doesn't make sense)
Since you do this modif, you may can do the same thing for all components with a web version. You missed this one: https://github.com/TERITORI/teritori-dapp/tree/68f8f691930c3a669e9cc037392694edacfdce30/packages/components/fileUploader
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which tool give you this error ? Because i have nothing that say that ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was happening when trying to build the project
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]>
Signed-off-by: clegirar <[email protected]> WIP: fix all unused exports from packages/networks Signed-off-by: clegirar <[email protected]>
d950096
to
f7f3666
Compare
If it's possible i want to rebase on main after merging Audio Player to have just one nightmare to resolve conflicts 🙃 |
Signed-off-by: clegirar <[email protected]>
f7f3666
to
97415b9
Compare
Signed-off-by: Norman Meier <[email protected]>
No description provided.