search

craustin / triply

Trip Planning Toolkit
http://desolate-thicket-4407.herokuapp.com/#/
3 stars 0 forks source link

Front page trip list pulls back entire database #7

Open craustin opened 10 years ago

craustin commented 10 years ago

It only needs to list the top-level keys. This will unnecessarily increase our Firebase data transfers.

ESRogs commented 10 years ago

What do you mean by keys? What's in the database that we don't show on the main page?

On Sat, Nov 23, 2013 at 1:22 PM, Craig Austin notifications@github.comwrote:

It only needs to list the top-level keys. This will unnecessarily increase our Firebase data transfers.

— Reply to this email directly or view it on GitHubhttps://github.com/craustin/triply/issues/7 .

craustin commented 10 years ago

Check out our Firebase: https://triply.firebaseio.com

The database contains a large JSON document containing costs and people for all trips. On the front page, I'm trying to show the first level of this JSON doc (keys of the top-level dict) - but the code is pulling the entire thing.

On Sat, Nov 23, 2013 at 5:08 PM, ESRogs notifications@github.com wrote:

What do you mean by keys? What's in the database that we don't show on the main page?

On Sat, Nov 23, 2013 at 1:22 PM, Craig Austin notifications@github.comwrote:

It only needs to list the top-level keys. This will unnecessarily increase our Firebase data transfers.

— Reply to this email directly or view it on GitHub< https://github.com/craustin/triply/issues/7> .

— Reply to this email directly or view it on GitHubhttps://github.com/craustin/triply/issues/7#issuecomment-29142919 .