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

Disable abilities extension-wide #3734

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Disable abilities extension-wide #3734

merged 1 commit into from
Feb 16, 2024

Conversation

Shadowfiend
Copy link
Contributor

@Shadowfiend Shadowfiend commented Feb 16, 2024

Due to the way that abilities are currently implemented, load on the Daylight API side is very high. We don't have the cycles right now to implement the underlying changes that would improve API load, so for now we disable abilities fetching, as well as access to abilities in the UI.

The code that is commented out is targeted to only remove the parts that link to the abilities UI and the parts that call out to the Daylight API; the rest remains in place so that turning things back on once the fetching strategy can be improved is straightforward.

Latest build: extension-builds-3734 (as of Fri, 16 Feb 2024 21:10:27 GMT).

Due to the way that abilities are currently implemented, load on the
Daylight API side is very high. We don't have the cycles right now to
implement the underlying changes that would improve API load, so for now
we disable abilities fetching, as well as access to abilities in the UI.

The code that is commented out is targeted to only remove the parts that
link to the abilities UI and the parts that call out to the Daylight
API; the rest remains in place so that turning things back on once the
fetching strategy can be improved is straightforward.
@Shadowfiend
Copy link
Contributor Author

Turboing this for release.

@Shadowfiend Shadowfiend merged commit 724e478 into main Feb 16, 2024
6 checks passed
@Shadowfiend Shadowfiend deleted the disable-abilities branch February 16, 2024 21:29
@Shadowfiend Shadowfiend mentioned this pull request Feb 16, 2024
Shadowfiend added a commit that referenced this pull request Feb 19, 2024
## What's Changed
* v0.56.0 by @Shadowfiend in
#3729
* Drop Arbitrum Sepolia from Alchemy supported network ids by
@Shadowfiend in #3730
* Disable abilities extension-wide by @Shadowfiend in
#3734


**Full Changelog**:
v0.56.0...v0.57.0

Latest build:
[extension-builds-3735](https://github.com/tahowallet/extension/suites/20840877238/artifacts/1252537126)
(as of Fri, 16 Feb 2024 22:05:55 GMT).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant