Skip to content
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion config/constants/addresses.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1228,7 +1228,9 @@ export const midasAddressesPerNetwork: ConfigPerNetwork<
},
eurc: {
token: '0x174d1A887e971f7d0fe5C68b328c30e0ED743160',
aggregator: '0x6CDbBb744a04740a179AF4C92E03C1Bf1eC552C8',
// Previously used: 0x6CDbBb744a04740a179AF4C92E03C1Bf1eC552C8 (manually managed oracle)
// Now using the oracle provided by the Scroll team: 0x24c8964338Deb5204B096039147B8e8C3AEa42Cc
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The new aggregator address is repeated in this comment and in the code on the next line. This is redundant and can lead to inconsistencies if the address is updated in one place but not the other. It's a good practice for comments to explain why a change was made, not what the code is doing, as the code itself shows the 'what'.

Suggested change
// Now using the oracle provided by the Scroll team: 0x24c8964338Deb5204B096039147B8e8C3AEa42Cc
// Now using the oracle provided by the Scroll team.

aggregator: '0x24c8964338Deb5204B096039147B8e8C3AEa42Cc',
dataFeed: '0x0c1E886A0ceBF805f66223333e204d39BE6612De',
},
},
Expand Down
Loading