EventLoader unnecessarily does an AJAX get for signed in users

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: peterbe, Assigned: lfresh)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
In starredevents.js it has::


    $(function() {
        EventLoader.load();
    });


This is not necessary for signed in people because they already have that page perfectly rendered on the server. Signed in people should of course use EventLoader.load() if they un-click a star as that'll reload the page in-place with ajax.

Comment 2

4 years ago
Commits pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/dbcbcf22fc00c17422b34a6dad6974b203f6423f
Fixes bug 1147902 - No initial EventLoader.load() for signed in

https://github.com/mozilla/airmozilla/commit/53a576b6ae2bf49647744bf39344bc5e9c659ba7
Merge pull request #255 from bugzPDX/1147902

Fixes bug 1147902 - No initial EventLoader.load() for signed in

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.