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.

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

Jump to bottom

Events Fragment Speed Improvment #51

Open
shreyasnbhat opened this issue Jan 31, 2017 · 2 comments
Open

Events Fragment Speed Improvment #51

shreyasnbhat opened this issue Jan 31, 2017 · 2 comments
Labels
bug

Comments

@shreyasnbhat
Copy link
Member

shreyasnbhat commented Jan 31, 2017 •
edited
Loading

Issue Fix

In onChildChanged method you could check current realm data with Firebase fetched data and then decide if you would want to update the existing entries.
This would avoid a lot of un-necessary updates to the Realm Database.
Figured this in the GoT-App as it had a lot of data and gave like 300 frames skipped for sorting and un-neccesary updates of data.

@kukreja-vikramaditya
kukreja-vikramaditya added a commit that referenced this issue Feb 23, 2017
@kukreja-vikramaditya
Added basic framework of IssuesActivity
7a948b3 
Fixed AlarmReceiver bug which caused UpdateCheckerService being called multiple times
performance increase in HeraldAdapter and EventsAdapter #51
ViewPagerAdapter now uses SparseArray instead of List
Methods enhanced in EventItem and HeraldItem
Gazettes back at ValueEventListener
Realm version bump
Minor fixes #39
@kukreja-vikramaditya
Copy link
Contributor

Will look into this soon

@kukreja-vikramaditya
Copy link
Contributor

Minor speed enhancement by updating EventItem format class in new branch commit 7a948b3

kukreja-vikramaditya added a commit that referenced this issue Feb 25, 2017
@kukreja-vikramaditya
New branch merge (#54)
51b4218 
PostDownloaderActivity improved
ImagesAndMedia update
Removed unused fields in HeraldItem (#14)
Deleted ImageDownloaderService
Deleted PostDownloadService
BootBroadcastReceiver updates only 1 page of Herald
UpdateCheckerService major updates (#4)
Reduced MAC logo size
Added placehoder for herald
removed Screen.jpg
reduced marker icon size
Gradle updates
Fixed UpdateCheckerService being called for first time (#4)
Added calendar intent and fixed other minor bugs
Fixed AlarmReceiver bug which caused UpdateCheckerService being called multiple times
performance increase in HeraldAdapter and EventsAdapter #51
ViewPagerAdapter now uses SparseArray instead of List
Methods enhanced in EventItem and HeraldItem
Gazettes back at ValueEventListener
Realm version bump
Minor fixes #39 
About Us Changes
Utilities now show more columns on orientation change
Added setTheme() in IssuesActivity
Fixed IssuesActivity
Fixed "currentItem" key in MainActivity
Fixed issue #46
Added a few method docs in DHC
Added collapsable toolbar in MainActivity (#53)
@Rushi98 Rushi98 removed the hard label May 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kukreja-vikramaditya @Rushi98 @shreyasnbhat