Fix the view command having the claimed by field when it doesn't need it (#292)
All checks were successful
Deploy To Stage / build (push) Successful in 9s
Deploy To Stage / deploy (push) Successful in 17s

- Remove the "claimed by" field from the card embed generator if its not supplied
- This does affect the drop command, now the drop command only shows the field upon claim, but I think that looks better

#276

Reviewed-on: #292
Co-authored-by: Ethan Lane <ethan@vylpes.com>
Co-committed-by: Ethan Lane <ethan@vylpes.com>
This commit is contained in:
Ethan Lane 2024-07-08 17:55:12 +01:00 committed by Vylpes
parent eb3b04f51c
commit 1360452ffd

View file

@ -94,13 +94,18 @@ export default class CardDropHelperMetadata {
name: "Claimed", name: "Claimed",
value: `${quantityClaimed}`, value: `${quantityClaimed}`,
inline: true, inline: true,
}, }
]);
if (claimedBy != null) {
embed.addFields([
{ {
name: "Claimed by", name: "Claimed by",
value: claimedBy ?? "(UNCLAIMED)", value: claimedBy,
inline: true, inline: true,
} }
]); ]);
}
if (currency != null) { if (currency != null) {
embed.addFields([ embed.addFields([