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.

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

feat(config): ability to customize "max sweep fee change" setting #793

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

theborakompanioni
Copy link
Collaborator

@theborakompanioni theborakompanioni commented Jul 9, 2024

Ability to customize the max_sweep_fee_change settings property.

Part of #761. This enables loading the settings value and showing the user a maximum fee for sweeps, e.g. in the confirm dialog.

📸


@theborakompanioni theborakompanioni added the enhancement New feature or request label Jul 9, 2024
@theborakompanioni theborakompanioni self-assigned this Jul 9, 2024
@theborakompanioni theborakompanioni marked this pull request as ready for review July 9, 2024 16:06
Copy link
Contributor

@MarnixCroes MarnixCroes left a comment

Choose a reason for hiding this comment

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

cACK

src/i18n/locales/en/translation.json Show resolved Hide resolved
@theborakompanioni theborakompanioni force-pushed the feat/max-sweep-fee-change branch from 7b137f0 to 243ab88 Compare July 9, 2024 16:27
@editwentyone
Copy link

addition: It would be nice to get this component into "send options" as soon as the user uses "Sweeps" function.

so, when the user prepared everything at the top to sweep, and opens up the send options, the user would get the max sweep fee component presented as a shortcut underneath the mining fee input field

@theborakompanioni
Copy link
Collaborator Author

addition: It would be nice to get this component into "send options" as soon as the user uses "Sweeps" function.

so, when the user prepared everything at the top to sweep, and opens up the send options, the user would get the max sweep fee component presented as a shortcut underneath the mining fee input field

Great idea!

@theborakompanioni theborakompanioni merged commit b4f8a56 into devel Jul 16, 2024
@theborakompanioni theborakompanioni deleted the feat/max-sweep-fee-change branch July 16, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants