Skip to content

[DO NOT MERGE] android: remove testnet argument #211

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

Closed
wants to merge 17 commits into from

Conversation

johnny9
Copy link
Collaborator

@johnny9 johnny9 commented Dec 15, 2022

ARM64 Android

jarolrod and others added 16 commits December 14, 2022 03:18
This does away with a loadable detail and places a check mark icon in
it's place.
Wires both the easy and advanced configuration options related to
storage amount to the options model backend. Additionally, this removes
an unused and unwanted storage location setting from the advanced
storage amount settings table.
- This method would allow to get the blocktime value of the block at
  the given height.
- This file allowing using of Chain interfaces to get information about
  the blockchain.
- This class declares and defines the function that will be used in
  for the Block Clock.
1. Added a new Q_PROPERTY to track the remaining time till the IBD
   completioni.
2. Added two signals to later connect to the chainmodel class function.
3. Added new member variable, m_pause, which will keep track of if the
   block clock is in paused state or not, and will accordingly
disconnect or reconnect the user to network.
- Also made appropriate connections with the nodemodel signals
- This property will later be used for the BlockClock.qml file
- The BlockClock.qml files provides the control for BlockClock and
  determines how the elements would be displayed.
- The BlockClockComponent.qml file uses the aforementioned control to
  display the value, and determines what will be displayed.
@johnny9 johnny9 marked this pull request as draft December 15, 2022 16:46
@johnny9
Copy link
Collaborator Author

johnny9 commented Dec 15, 2022

@jarolrod
Copy link
Member

For context this is here because bitcoin/bitcoin#17227 introduced it when introducing android packaging support, it mainly seems because this initial android support was to be treated as experimental. At some point we'll change this though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants