Skip to content

[DO NOT MERGE] Review current state#267

Closed
jarolrod wants to merge 20 commits into
bitcoin-core:mainfrom
jarolrod:review-current-state
Closed

[DO NOT MERGE] Review current state#267
jarolrod wants to merge 20 commits into
bitcoin-core:mainfrom
jarolrod:review-current-state

Conversation

@jarolrod
Copy link
Copy Markdown
Contributor

@jarolrod jarolrod commented Feb 17, 2023

This includes #223, #245, #256, #257, #259, #261, and #264

This is for convenient testing for the scheduled call (Core App Call #50)

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

jarolrod and others added 20 commits February 17, 2023 02:46
SettingsConnection now contains a SwipeView where you can access the
connection settings then press on "Proxy Settings" to get to the Proxy
configuration page. There is no use-case for Proxy settings to be shown
outside of the context of connection settings.
This allows for a new OptionButton to be shown in StorageOptions when
the user has going into detailed settings during onboarding and chosen
a custom prune target
The EMPTY state indicates that this setting hasn't been touched.
Clicking on the Setting that contains the ValueInput is enough to put
it in the FILLED state.
Adds an IntValidator to integer input ValueInput settings. Additionally,
specifies the "errorText" for these settings as well as the condition
for when to load the errorText
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.

2 participants