Skip to content

Conversation

@M-DinhHoangViet
Copy link
Contributor

No description provided.

tr(unlimited)(
"Play and create ",
a(href := routes.Simul.home)("simultaneous exhibitions")
a(href := routes.Tournament.home)(trans.features.playAndCreateSimul())
Copy link
Collaborator

Choose a reason for hiding this comment

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

you changed the href and made it wrong.

Copy link
Collaborator

@ornicar ornicar Jul 15, 2024

Choose a reason for hiding this comment

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

this translation key is singular while the previous one is plural, fortunately they're both superfluous

<string name="ifYouLoveLichess">If you love Lichess,</string>
<string name="supportUsWithAPatronAccount">Support us with a Patron account!</string>
<string name="allFeaturesAreFreeForEverybody">all features are free for everybody, forever!</string>
<string name="weBelieveEveryChessPlayerDeservesTheBest">We believe every draughts player deserves the best, and so:</string>
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this a joke?

a(cls := "button", href := routes.Plan.index)("Support us with a Patron account!")
trans.features.ifYouLoveLichess(),
a(cls := "button", href := routes.Plan.index)(trans.features.supportUsWithAPatronAccount())
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

the text of this box no longer makes sense.

@ornicar
Copy link
Collaborator

ornicar commented Jul 16, 2024

Your pull requests take a long time to review and fix.

Please try to focus more on quality and less on quantity. Your motivation should be to help the project rather than feeding your fancy github profile.

image

So please take the time to proof-read your code, run it, and adhere to standards. Instead of relying on me to do it for you.

@ornicar ornicar merged commit 1113758 into lichess-org:master Jul 16, 2024
@M-DinhHoangViet
Copy link
Contributor Author

Hi, I just spent a wonderful summer vacation and delayed this PR, I'm sorry for not being put as a draft

@M-DinhHoangViet M-DinhHoangViet deleted the missing-trans-features-page branch July 17, 2024 11:52
@M-DinhHoangViet M-DinhHoangViet restored the missing-trans-features-page branch July 17, 2024 11:52
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