Skip to content

Notifications #197

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.

Sign up for GitHub

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

Merged
camilasan merged 22 commits into from
Mar 13, 2018
Merged

Notifications #197

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
a7dd3a7
Displays the actual notification subject in the system tray messages.
Feb 12, 2018
b24751b
Adds button to open notification in the browser - to update something…
Feb 14, 2018
bfc7aa1
Adds option to toggle display of Sync and Activity notifications.
Feb 14, 2018
ab7185a
Changes notification text in the checkboxes and adds tooltips with ex…
Feb 15, 2018
20cf4ed
Improves Notification Widget UI.
Feb 16, 2018
32e29fb
Doesn't open the file manager when there isn't a file associated with…
Feb 16, 2018
1a0edc1
Sync activities notifications are never shown unless there is an erro…
Feb 23, 2018
4c79059
Changes notification icon bell size.
Feb 25, 2018
e33d2bf
Removes (s) from time strings.
Feb 25, 2018
4a6a8ca
Removes _appName from Activity data.
Feb 25, 2018
71f4760
Do not display user account when there is only one user in popup mess…
Feb 25, 2018
d71438f
Adds host and scheme to activity link so it can be opened in the
Feb 25, 2018
44f299e
Removes debug messages.
Feb 25, 2018
0f9d8e1
Only uses 'More information' text in button for notification link.
Feb 25, 2018
09a92a5
Align text and changes spacing in notification widget.
Feb 25, 2018
112c5e1
Removes Server Activities option from General settings.
Feb 26, 2018
56709f3
Removes string 'Created' from notifications time.
Feb 26, 2018
6bf9f65
Only notify users of new notifications and only shows them all on sta…
Feb 27, 2018
d42401e
Changes notifications api endpoint to v2.
Mar 1, 2018
621596f
Saves notifications ETag response header to use to request only new n…
Mar 1, 2018
3c77898
Makes sure JsonApiJob::finished won't throw a JSON error when status …
Mar 3, 2018
2e4472a
Enables server notifications by default.
Mar 3, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Do not display user account when there is only one user in popup mess… 
…age.

Signed-off-by: Camila San <[email protected]>
  • Loading branch information
Camila San committed Feb 25, 2018
commit 71f47602152b4232032045f3618fd01faebdafe6
11 changes: 6 additions & 5 deletions src/gui/activitywidget.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,8 +249,6 @@ void ActivityWidget::slotBuildNotificationDisplay(const ActivityList &list)
bool newNotificationShown = false;

foreach (auto activity, list) {
qDebug() << "Notification list"
<< activity._subject;
if (_blacklistedNotifications.contains(activity)) {
qCInfo(lcActivity) << "Activity in blacklist, skip";
continue;
Expand Down Expand Up @@ -311,9 +309,12 @@ void ActivityWidget::slotBuildNotificationDisplay(const ActivityList &list)
// Assemble a tray notification
ConfigFile cfg;
if(cfg.optionalServerNotifications()){
QString log = tr("Notification for %1.").arg(activity._accName);
emit guiLog(log, activity._subject);
}
if(AccountManager::instance()->accounts().count() == 1){
emit guiLog(activity._subject, "");
} else {
emit guiLog(activity._subject, activity._accName);
}
}
}

// check if there are widgets that have no corresponding activity from
Expand Down