Skip to content

fix: make metadata mento specific #39

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.

Sign up for GitHub

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

Open
philbow61 wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

fix: make metadata mento specific #39

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
43fc73b
fix: make metadata mento specific
philbow61 Feb 23, 2026
f0053b9
chore: update NFT metadata
philbow61 Feb 24, 2026
ee89eab
fix: NFT metadata test
philbow61 Feb 24, 2026
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
8 changes: 5 additions & 3 deletions contracts/src/NFTMetadata/MetadataNFT.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,13 @@ contract MetadataNFT is IMetadataNFT {
function uri(TroveData memory _troveData) public view returns (string memory) {
string memory attr = attributes(_troveData);
return json.formattedMetadata(
string.concat("Liquity V2 - ", IERC20Metadata(_troveData._collToken).name()),
string.concat("Mento Liquity V2 - ", IERC20Metadata(_troveData._collToken).name()),
string.concat(
"Liquity V2 is a collateralized debt platform. Users can lock up ",
"Mento Liquity V2 is a collateralized debt platform. Users can lock up ",
IERC20Metadata(_troveData._collToken).symbol(),
" to issue stablecoin tokens (BOLD) to their own Ethereum address. The individual collateralized debt positions are called Troves, and are represented as NFTs."
" to issue ",
IERC20Metadata(_troveData._boldToken).symbol(),
" to their own address. The individual collateralized debt positions are called Troves, and are represented as NFTs."
),
renderSVGImage(_troveData),
attr
Expand Down
11 changes: 2 additions & 9 deletions contracts/src/NFTMetadata/utils/bauhaus.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,8 @@ library bauhaus {
function _bauhaus(string memory _collName, uint256 _troveId) internal pure returns (string memory) {
bytes32 collSig = keccak256(bytes(_collName));
uint256 variant = _troveId % 4;

if (collSig == keccak256("WETH")) {
return _img1(variant);
} else if (collSig == keccak256("wstETH")) {
return _img2(variant);
} else {
// assume rETH
return _img3(variant);
}
// Always use _img1 because in MentoV3 we only have one collateral token
return _img1(variant);
}

function _colorCode2Hex(colorCode _color) private pure returns (string memory) {
Expand Down
4 changes: 2 additions & 2 deletions contracts/test/troveNFT.t.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,11 +201,11 @@ contract troveNFTTest is DevTestSetup {
string memory decodedUri = string(Base64.decode(uriSplit));

// Check for expected attributes
assertTrue(LibString.contains(decodedUri, '"name": "Liquity V2 - '), "NFT Name attribute missing");
assertTrue(LibString.contains(decodedUri, '"name": "Mento Liquity V2 - '), "NFT Name attribute missing");

assertTrue(
LibString.contains(
decodedUri, '"description": "Liquity V2 is a collateralized debt platform. Users can lock up'
decodedUri, '"description": "Mento Liquity V2 is a collateralized debt platform. Users can lock up'
),
"NFT description attribute missing"
);
Expand Down