Closed Bug 1147902 Opened 9 years ago Closed 9 years ago

EventLoader unnecessarily does an AJAX get for signed in users

Categories

(Webtools Graveyard :: Air Mozilla, defect)

Product:
Webtools Graveyard
Component:
Air Mozilla
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: peterbe, Assigned: lfresh)

Details

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.
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
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.