archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

Notifications: Regulate fetching via cookie

Browse Source 
We introduce a doNotQueryNotifications cookie with a short lifetime,
which is used to determine wether getNotifications should be called
or not. This prevents notifications from being fetched at every page
load, unless the cookie is expired.
This commit is contained in:
Francesco Siddi 2020-04-17 13:32:22 +02:00
parent 86206d42dc
commit 5ab4086cbe
1 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/scripts/tutti/5_1_notifications.js
View File

@ -1,5 +1,3 @@
/**
* Store the number of unread notifications on load.
* That way, if the number got higher while the page was
@ -19,8 +17,24 @@ function clearNotificationIcon(){
}
// Get notifications by fetching /notifications/ JSON every 30 seconds
/**
* Get notifications by fetching /notifications/ and update ul#notifications-list
*
* This is called every 60 seconds by getNotificationsLoop() but the endpoint is queried only if the
* doNotQueryNotifications cookie is expired. If so, the cookie is created again, with a lifetime
* of 65 seconds.
*/
function getNotifications(){
//- Check if the cookie is still valid, in which case, return
if( Cookies('doNotQueryNotifications') != null ) {return;}
//- Create a new cookie, which expires in 65 seconds
Cookies.set('doNotQueryNotifications', 'true', {
expires: new Date(new Date().getTime() + 65 * 1000)
});
//- Fetch data and update the interface
$.getJSON( "/notifications/", function( data ) {
if (!first_load) {