-
Notifications
You must be signed in to change notification settings - Fork 21
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 ST v1 #1748
chore: remove ST v1 #1748
Conversation
@limpbrains I wonder if we should just merge this PR, test the performance of the app, to confirm that the DHT is causing most of the issues, then consider our options to add slashauth again in the light of that expirement. |
Good idea, I will test it tomorrow |
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
53c1752
to
4740ef3
Compare
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.
Tested and LGTM
Co-authored-by: Philipp Walter <[email protected]>
Description
I will squash commits after review
Linked Issues/Tasks
Type of change
Refactoring
Tests
No test
QA Notes
Install prev version, add slashauth, upgrade, try to use slashauth. Widget should disapear