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 plugin update notifications. #620

Merged
merged 5 commits into from Aug 13, 2019

Conversation

@th-h
Copy link
Member

th-h commented Aug 10, 2019

First pass.

  • Count available plugin upgrades.
  • Add number of updates to plugin upgrade button.
    (Could use some improvements, a badge perhaps?)
  • Add upgrade notification to dashboard.
    (Could use some polish, too.)

Tested on our "current" test blog, seems to work.

Fixes #423.

th-h added 5 commits Aug 10, 2019
New function count_plugin_upgrades() to count
upgradeable plugins, sidebar as well as event.

Signed-off-by: Thomas Hochstein <thh@inter.net>
This could use a bit of eye candy.

Signed-off-by: Thomas Hochstein <thh@inter.net>
New 'backend_plugins_upgradecount' hook.
Add language constants.

Signed-off-by: Thomas Hochstein <thh@inter.net>
Signed-off-by: Thomas Hochstein <thh@inter.net>
Signed-off-by: Thomas Hochstein <thh@inter.net>
@th-h th-h requested review from yellowled and onli Aug 10, 2019
@onli
onli approved these changes Aug 10, 2019
@garvinhicking

This comment has been minimized.

Copy link
Member

garvinhicking commented Aug 13, 2019

Cool! Gefällt mir!

@yellowled

This comment has been minimized.

Copy link
Member

yellowled commented Aug 13, 2019

Ich glaube <hr class="separator"> ist zumindest an dieser Stelle (innerhalb des {if}) falsch, das dürfte nur ausgegeben werden, wenn in einer „Reihe“ im Dashboard zwei Widgets ausgegeben werden (was so bisher, glaube ich, einfach nicht vorgesehen war). Und zudem müssten dann wohl auch <section id="dashboard_updates"> und <section id="dashboard_plugin_updates"> die zusätzlichen Klassen (class="equal_heights quick_list dashboard_widget") bekommen, damit das alles etwas weniger „windschief“ aussieht.

Eventuell ist es auch einfach mal an der Zeit, sich diesen ganzen Kram mal wieder genauer anzugucken. (Also: für mich.)

Ich sehe jedenfalls keinen Grund, das nicht so oder so zu mergen. Passt die Box halt nicht genau, who cares? Die Funktionalität ist zu cool.

@th-h

This comment has been minimized.

Copy link
Member Author

th-h commented Aug 13, 2019

Ich glaube <hr class="separator"> ist zumindest an dieser Stelle (innerhalb des {if}) falsch, das dürfte nur ausgegeben werden, wenn in einer „Reihe“ im Dashboard zwei Widgets ausgegeben werden (was so bisher, glaube ich, einfach nicht vorgesehen war). Und zudem müssten dann wohl auch <section id="dashboard_updates"> und <section id="dashboard_plugin_updates"> die zusätzlichen Klassen (class="equal_heights quick_list dashboard_widget") bekommen, damit das alles etwas weniger „windschief“ aussieht.

Ja, irgendwas stimmt da nicht; das Widget für Kommentare glaubt nämlich, es stünde rechts, so das die Abstände nicht stimmen. Ich habe den Code aber im Prinzip per c&p von dem Update-Hinweis für das Blog übernommen. Der müsste dann eigentlich genauso windschief sein ...

Ich würde es aber trotzdem erstmal mergen, bis Du vielleicht nochmal genauer draufgucken kannst, wie das CSS richtig aussehen müsste.

@th-h th-h merged commit 221bd4a into s9y:master Aug 13, 2019
@yellowled

This comment has been minimized.

Copy link
Member

yellowled commented Aug 13, 2019

Ich habe den Code aber im Prinzip per c&p von dem Update-Hinweis für das Blog übernommen. Der müsste dann eigentlich genauso windschief sein ...

Ich glaube, der geht einfach davon aus, dass ihm ggf. die ganze Zeile gehört, weil es sonst keine weiteren Dashboard-Hinweise gab.

@th-h

This comment has been minimized.

Copy link
Member Author

th-h commented Aug 13, 2019

Tracked in #624

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.