Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

  • Make sure the translation command for "Offline" can be parsed. Otherwise it will not be extracted as a translation entry.
  • Use lastSave as the variable name in the tooltip. That is what all the translations use right now.

Fixes #3750.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests. We currently do not test translations. Might be worthwhile though as this was a regression.
  • Documentation is not required

@max-nextcloud
Copy link
Collaborator Author

/compile

@max-nextcloud max-nextcloud requested review from a team, luka-nextcloud and marcelklehr and removed request for a team February 6, 2023 06:56
@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Feb 6, 2023
@juliusknorr juliusknorr added this to the Nextcloud 26 milestone Feb 6, 2023
* Make sure the translation command for "Offline" can be parsed.
  Otherwise it will not be extracted as a translation entry.
* Use `lastSave` as the variable name in the tooltip.
  That is what all the translations use right now.

Fixes #3750.

Signed-off-by: Max <[email protected]>
@max-nextcloud max-nextcloud force-pushed the fix/translations-in-status branch from 20de00d to d055707 Compare February 6, 2023 13:44
@max-nextcloud
Copy link
Collaborator Author

/compile

@cypress
Copy link

cypress bot commented Feb 6, 2023

Passing run #8486 ↗︎

0 132 0 0 Flakiness 0

Details:

fix: transations in status message
Project: Text Commit: b537208129
Status: Passed Duration: 05:49 💡
Started: Feb 6, 2023 2:03 PM Ended: Feb 6, 2023 2:08 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Signed-off-by: nextcloud-command <[email protected]>
@max-nextcloud
Copy link
Collaborator Author

/backport to stable25

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

Labels

4. to release bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The offline warning seams to be hard-coded (cannont be translated) and the "lastSave" information is not working

5 participants