Skip to content

Conversation

@PastaPastaPasta
Copy link
Member

@PastaPastaPasta PastaPastaPasta commented Jun 27, 2020

I wasn't sure if I should do the second commit manually, or if that'd happen automatically so I did it in a second commit so I can easily revert. reverted. I'm 90% sure.

I think this makes litemode a lot clearer and less confusing.

TODO:
I need to add a check and throw an error if someone uses litemode or just interpret litemode the same as disablegovernance

@PastaPastaPasta PastaPastaPasta added this to the 16 milestone Jun 27, 2020
@PastaPastaPasta PastaPastaPasta force-pushed the change-litemode-disablegovernance branch from 1ac807a to b5a8fb7 Compare June 27, 2020 23:35
@PastaPastaPasta
Copy link
Member Author

Ready for review. Tested locally. Seems to be working as expected

UdjinM6
UdjinM6 previously approved these changes Jun 28, 2020
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@PastaPastaPasta PastaPastaPasta force-pushed the change-litemode-disablegovernance branch from 15ffe57 to 72bdaa7 Compare June 28, 2020 15:33
@PastaPastaPasta
Copy link
Member Author

Accidentally added two commits meant for a later PR. dropped those commits and rebased on develop

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-utACK

@UdjinM6 UdjinM6 merged commit b463e57 into dashpay:develop Jun 28, 2020
@PastaPastaPasta PastaPastaPasta deleted the change-litemode-disablegovernance branch June 29, 2020 03:41
PastaPastaPasta added a commit to PastaPastaPasta/dash that referenced this pull request Aug 23, 2020
* Change litemode to disablegovernance, this makes it more clear what that flag actually does.

Signed-off-by: pasta <[email protected]>

* more fLiteMode to fDisableGovernance

Signed-off-by: pasta <[email protected]>

* handle -litemode being set

Signed-off-by: pasta <[email protected]>
@PastaPastaPasta
Copy link
Member Author

backported in #3670

gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 8, 2022
* Change litemode to disablegovernance, this makes it more clear what that flag actually does.

Signed-off-by: pasta <[email protected]>

* more fLiteMode to fDisableGovernance

Signed-off-by: pasta <[email protected]>

* handle -litemode being set

Signed-off-by: pasta <[email protected]>
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