apollo_consensus_orchestrator: rename snip35 module to dynamic_gas_price, scrub SNIP-35 comments#14216
Conversation
matanl-starkware
left a comment
There was a problem hiding this comment.
@matanl-starkware partially reviewed 26 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on sirandreww-starkware).
crates/apollo_dashboard/src/panels/consensus.rs line 726 at r1 (raw file):
fn get_panel_snip35_fee_actual() -> Panel { Panel::new( "SNIP-35 - fee_actual (GFri)",
Remove this prefix from all the panels, it's already under the SNIP-35 row.
Code quote:
SNIP-35 - crates/apollo_dashboard/src/panels/consensus.rs line 797 at r1 (raw file):
pub(crate) fn get_snip35_row() -> Row { Row::new( "SNIP-35 - Dynamic Gas Price",
Suggestion:
Dynamic Gas Price (SNIP-35)
PR SummaryLow Risk Overview Behavior is unchanged. Fee math, validation, and wire formats stay the same; only labels move from “SNIP-35” toward neutral terms (e.g. Grafana/dashboard copy is refreshed (row “Dynamic Gas Price (SNIP-35)”, shorter panel titles); Prometheus metric names such as Reviewed by Cursor Bugbot for commit 8d2400d. Bugbot is set up for automated code reviews on this repo. Configure here. |
56bcecd to
1c2514e
Compare
477d3d8 to
9d8027d
Compare
sirandreww-starkware
left a comment
There was a problem hiding this comment.
@sirandreww-starkware made 2 comments.
Reviewable status: 24 of 26 files reviewed, 2 unresolved discussions (waiting on matanl-starkware).
crates/apollo_dashboard/src/panels/consensus.rs line 726 at r1 (raw file):
Previously, matanl-starkware (Matan Lior) wrote…
Remove this prefix from all the panels, it's already under the SNIP-35 row.
Done.
crates/apollo_dashboard/src/panels/consensus.rs line 797 at r1 (raw file):
pub(crate) fn get_snip35_row() -> Row { Row::new( "SNIP-35 - Dynamic Gas Price",
Done.
matanl-starkware
left a comment
There was a problem hiding this comment.
@matanl-starkware reviewed 2 files and all commit messages, made 1 comment, and resolved 2 discussions.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).
crates/apollo_dashboard/src/panels/consensus.rs line 726 at r2 (raw file):
fn get_panel_snip35_fee_actual() -> Panel { Panel::new( "fee_actual (GFri)",
All panels should have the same style (Title Case).
e.g. "Seconds Since Last Successful STRK→USD Rate Update"
Code quote:
fee_actual (GFri)
matanl-starkware
left a comment
There was a problem hiding this comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).
9d8027d to
30aed32
Compare
…ice, scrub SNIP-35 comments
30aed32 to
8d2400d
Compare
sirandreww-starkware
left a comment
There was a problem hiding this comment.
@sirandreww-starkware made 1 comment and resolved 1 discussion.
Reviewable status: 23 of 26 files reviewed, all discussions resolved (waiting on matanl-starkware).
crates/apollo_dashboard/src/panels/consensus.rs line 726 at r2 (raw file):
Previously, matanl-starkware (Matan Lior) wrote…
All panels should have the same style (Title Case).
e.g. "Seconds Since Last Successful STRK→USD Rate Update"
Done
sirandreww-starkware
left a comment
There was a problem hiding this comment.
@sirandreww-starkware partially reviewed 26 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on sirandreww-starkware).

No description provided.