fix(renterd): config fields blank sia central

[alexfreska]

• Fixed an issue where configured settings would show as blank when the Sia Central API (used for network averages) was down. Autopilot settings are not visible in UI when Siacentral API is down renterd#1601
• Fixed an issue where SiacoinField was depending on the siaCentral setting rather than the active exchange rate API.
• Refactoring
  • Refactored the internal autopilotInfo usage to be more clear.
  • Refactored useResources and resources into one file.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
renterd	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 10, 2024 9:32pm

4 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
explorer	⬜️ Ignored (Inspect)	Visit Preview		Oct 10, 2024 9:32pm
explorer-zen	⬜️ Ignored (Inspect)	Visit Preview		Oct 10, 2024 9:32pm
hostd	⬜️ Ignored (Inspect)	Visit Preview		Oct 10, 2024 9:32pm
website	⬜️ Ignored (Inspect)	Visit Preview		Oct 10, 2024 9:32pm

[changeset-bot]

🦋 Changeset detected

Latest commit: d6b71d7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
renterd	Patch
@siafoundation/design-system	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[alexfreska]

• fix(renterd): config fields blank sia central #767 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alexfreska and the rest of your teammates on Graphite

[telestrial]

If I understand the change correctly: do we currently tell the user that the API is down? They may have expectations about what they're seeing and be confused when they don't see what they expect.

[alexfreska]

If I understand the change correctly: do we currently tell the user that the API is down? They may have expectations about what they're seeing and be confused when they don't see what they expect.

@telestrial SiacoinFields (the ones that show both siacoin input and the fiat input below) gracefully degrade, meaning they do not show the fiat input if the exchange rate is not available.
The renterd pinned configuration fields get completely disabled and show the user a warning message stating that the API is required for the feature.

It would probably be a good improvement to show a similar warning on SiacoinField. I'll create an issue: #777

[alexfreska]

Merge activity

• Oct 11, 1:53 PM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 11, 1:53 PM EDT: A user merged this pull request with Graphite.