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

Address (Windows) compile warnings #840

Merged
merged 5 commits into from
Jun 28, 2020

Conversation

Lord-Grey
Copy link
Collaborator

@Lord-Grey Lord-Grey commented Jun 21, 2020

Summary

PR address multiple (Windows) compile (Qt 5.15 deprecation) warnings.
These warnings are not windows specific and are addressed generically.

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing setups:

The PR fulfills these requirements:

  • When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature
  • Related documents have been updated (docs/docs/en)
  • Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@Lord-Grey Lord-Grey changed the title Address Windows compile warnings Address (Windows) compile warnings Jun 21, 2020
@brindosch
Copy link
Contributor

Super,
da kommen wir nochmal zum Thema translations. Ich würde vermutlich nur EN updaten terms+strings. Die anderen Sprachen kann man dann als fuzzy deklarieren und benötigen ein Review der Übersetzer.

Ein automatisches Update muss mit jeder Sprache einzeln verlinkt werden und ich weiß nicht wie gut das funktioniert (Das keine falschen Übersetzungen überschrieben werden). Müsste man mal ein Test machen wie es sich verhält oder obige Lösung verwenden.

Ob "Fuzzy" von Übersetzer überhaupt wahrgenommen wird ist unbekannt. Ein leerer String (term umbenennen) in jedem Fall. Ggf mehr Doku nötig

@Lord-Grey
Copy link
Collaborator Author

@brindosch Hi, in diesem Fall habe ich auch nur die anderen Übersetzungen angepasst, da Hyperion durch Instanz zu ersetzten war und die Sprachen alle dem gleichen Muster folgen.
Ansonsten gebe ich Dir recht, dass mir nicht klar ist, wie der Prozess wäre, wenn ich Englisch aktualisiere und was dann bei den anderen Sprachen passieren muss (wahrscheinlich muss ich sie "blank" setzen"...)

@brindosch
Copy link
Contributor

Das ist alles in Ordnung. Es geht darum das nach aktuellem Stand deine zusätzlich Arbeit mit dem nächsten Export weg wäre.

@Lord-Grey
Copy link
Collaborator Author

Lord-Grey commented Jun 26, 2020

Ok. Danke. Das beutet, dass neue Items über mehrere Sprachen auch nicht funktionieren?

@brindosch
Copy link
Contributor

So, hab das eben mal getestet und ich muss leider bestätigen der import überschreibt alles. Also entweder mit POEditor übersetzen oder hier.

Oder ständig exportieren vor jedem commit. Das ist aber auch nicht praktisch. Über die POEditor api könnte man wahrscheinlich von Hand selektiv translations pushen, wenn ich das richtig gesehen habe. Da müsste man dann aus den commits entsprechend diffs aus den files rausarbeiten.

@hyperion-project
Copy link

Here is your new link to your workflow artifacts.

@brindosch
Copy link
Contributor

Ich hab die Änderungen in POEditor eingepflegt und die anderen Sprachen für den String zurückgesetzt.

Thanks for your contribution!

@brindosch brindosch merged commit bfb50b8 into hyperion-project:master Jun 28, 2020
@Lord-Grey Lord-Grey deleted the Fix_Warnings branch August 4, 2020 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants