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.

Sign up for GitHub

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

Jump to bottom

Remove pricetable endpoint from worker and bus #1635

Closed
ChrisSchinnerl opened this issue Oct 30, 2024 · 0 comments · Fixed by #1645
Closed

Remove pricetable endpoint from worker and bus #1635

ChrisSchinnerl opened this issue Oct 30, 2024 · 0 comments · Fixed by #1645
Assignees
@ChrisSchinnerl

Comments

@ChrisSchinnerl
Copy link
Member

ChrisSchinnerl commented Oct 30, 2024 •
edited
Loading

Pricetables can be fetched by the worker directly and should belong to the worker. The bus can keep unpaid price tables from scanning for the autopilot and the UI but the valid price tables can be handled by the worker only.

With rhp4 fetching a pricetable will be free which will simplify things further.
@ChrisSchinnerl ChrisSchinnerl converted this from a draft issue Oct 30, 2024
@ChrisSchinnerl ChrisSchinnerl moved this from Triage to Todo in Sia Oct 30, 2024
@ChrisSchinnerl ChrisSchinnerl mentioned this issue Oct 30, 2024
Open
@ChrisSchinnerl ChrisSchinnerl self-assigned this Oct 31, 2024
@ChrisSchinnerl ChrisSchinnerl changed the title Move PriceTable fetching to bus API Remove pricetable endpoint from worker and bus Oct 31, 2024
@ChrisSchinnerl ChrisSchinnerl mentioned this issue Oct 31, 2024
Merged
@github-project-automation github-project-automation bot moved this from Todo to Done in Sia Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChrisSchinnerl ChrisSchinnerl

Labels
None yet
Projects
Sia
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove pricetable endpoints SiaFoundation/renterd
1 participant
@ChrisSchinnerl