-
Notifications
You must be signed in to change notification settings - Fork 17
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
Update notification doesn't include package name and version [Bug] #300
Comments
Thank you very much @spodermenpls for this excellent bug report. I will take a closer look at the corresponding code, when I find some time for SailfishOS-related things (but not the next couple of days, unfortunately). |
Oops, I intended to do the same what you did now: Look at this issue when update candidates exist, respectively when updating them. Unfortunately I forgot, hence thank you for reminding me of this issue. I now tried to understand the code location, for which you nicely provided a link in the OP. Now I wonder, too: Why is not the usual scheme for emitting notifications used (as e.g. here), and where is the corresponding @piggz, as you authored this code (though three years ago), can you please take a look at this issue. The SailfishOS:Chum GUI app versions 0.6.9 and 0.7.0 bear no change in this regard (the OP addresses v0.6.8). Mind that in above screenshots only the notification bar at the very top of the screen is relevant (i.e. the topic of this issue). Thanks! |
@piggz, sorry to have stubbornly assigned this issue to you, as I assume this still is an less-annoying way to get your attention (and within the functionality of GH's web-frontend) than receiving a PM. Anyway, this issue can be re- or un-assigned at any time if you wish. |
Sailfish OS VERSION: 4.6.0.15 and earlier
HARDWARE: Sony Xperia XA2
SailfishOS:Chum GUI application VERSION: 0.6.8-1 and earlier
BUG DESCRIPTION
According to the locale string
%1 %2 updated
,%1
is supposed to act as a placeholder for the package name and%2
as a placeholder for the package version, that get displayed in a notification popup after a successful package update. However, package name and version never get displayed on my end, and have never done so (at least to my recollection) in the past either, it only displays "updated", or in my German case "aktualisiert" (which was the original thing that made me pay attention, since if indeed intended as a single word, it should be spelled with a capital "A" in that scenario). The notifications after Install or Remove operations however do include package name and version, as intended.STEPS TO REPRODUCE
If there's at least one update for a package available (unfortunately not that easy to re-create on demand..), execute it from the pulley menu and observe the notification pop-up banner after it is done.
ADDITIONAL INFORMATION
As mentioned in #167:
Affected strings in source TS file:
https://github.com/sailfishos-chum/sailfishos-chum-gui/blob/main/translations/sailfishos-chum-gui.ts#L39-L56
Corresponding code:
https://github.com/sailfishos-chum/sailfishos-chum-gui/blob/main/qml/sailfishos-chum-gui.qml#L13-L30
[Please consider which other pieces of information may be relevant: Denote if this is not always reproducible, if this is a regression (then name to which older version), attach relevant data such as log files or the systemd journal, provide screenshots etc.]
The text was updated successfully, but these errors were encountered: