Skip to content
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.

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

Add extra text formatter to abilities and upgrades #262

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

KingDarBoja
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

🌐 Branch deployed as preview to:
https://coh-stats-coh3-stats-extra-formatter-text.alt.free.edgio-perma.link

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Device URL
desktop https://preview.coh3stats.com

Device URL
mobile https://preview.coh3stats.com

Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead

Copy link
Member

@petrvecera petrvecera left a comment

Choose a reason for hiding this comment

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

helpText: string;
briefText: string;
screenName: string;
extraText: string; // Could be empty (Set as $0).
Copy link
Member

Choose a reason for hiding this comment

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

This is one thing I am not sure about.

image

I added these checks that if it's null it should not be displayed, so we can be sure what

if("") is false, so it's going to work, but it's something we can think off how to handle across the app.

Copy link
Collaborator Author

@KingDarBoja KingDarBoja Oct 5, 2023

Choose a reason for hiding this comment

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

Yeah, it does work without issues for both null or empty string checks. :)

@petrvecera
Copy link
Member

Nice 👍 ❤️

@petrvecera petrvecera merged commit a0ca06b into master Oct 5, 2023
4 checks passed
@KingDarBoja KingDarBoja deleted the extra-formatter-text branch October 5, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants