Skip to content

Comments

Add patron colored wings#2227

Merged
veloce merged 2 commits intomainfrom
patron_tiers
Sep 30, 2025
Merged

Add patron colored wings#2227
veloce merged 2 commits intomainfrom
patron_tiers

Conversation

@veloce
Copy link
Contributor

@veloce veloce commented Sep 26, 2025

No description provided.

@veloce veloce marked this pull request as ready for review September 30, 2025 00:25
@veloce veloce merged commit 1342598 into main Sep 30, 2025
1 check passed
@veloce veloce deleted the patron_tiers branch September 30, 2025 00:26
name: username,
title: title,
flair: flair,
isPatron: patron,
Copy link
Contributor

@ornicar ornicar Sep 30, 2025

Choose a reason for hiding this comment

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

this field is only here for BC. You don't need it, and should only use patronTierColor.

isPatron == patronColor.isDefined

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok will change that 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also Lila-ws has not been deployed it seems, cause patronColors is missing in online friends WS API.

Copy link
Contributor

Choose a reason for hiding this comment

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

Indeed and it might be another week before it's deployed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants