Skip to content

Commit

Permalink
Remove FeePriority::Default
Browse files Browse the repository at this point in the history
Done in consultation with @j-berman.

The RPC/CLI/GUI almost always adjust up except barring very explicit commands,
hence why FeePriority 0 is now only exposed via the explicit command of
FeePriority::Custom { priority: 0 }.

Also helps with terminology.
  • Loading branch information
kayabaNerve committed Feb 20, 2024
1 parent 1df4b8c commit e911415
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions coins/monero/src/wallet/send/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,6 @@ impl Fee {
#[derive(Clone, Copy, PartialEq, Eq, Debug)]
#[allow(non_camel_case_types)]
pub enum FeePriority {
Default,
Unimportant,
Normal,
Elevated,
Expand All @@ -283,7 +282,6 @@ pub enum FeePriority {
impl FeePriority {
pub(crate) fn fee_priority(&self) -> u32 {
match self {
FeePriority::Default => 0,
FeePriority::Unimportant => 1,
FeePriority::Normal => 2,
FeePriority::Elevated => 3,
Expand Down
2 changes: 0 additions & 2 deletions coins/monero/tests/wallet2_compatibility.rs
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ async fn from_wallet_rpc_to_self(spec: AddressSpec) {

// TODO: Needs https://github.com/monero-project/monero/pull/8882
// let fee_rate = daemon_rpc
// // Uses `FeePriority::Unimportant` instead of `FeePriority::Default` because wallet RPC
// // adjusts `monero_rpc::TransferPriority::Default` up by 1
// .get_fee(daemon_rpc.get_protocol().await.unwrap(), FeePriority::Unimportant)
// .await
// .unwrap();
Expand Down

0 comments on commit e911415

Please sign in to comment.