Email panel causes Admin instances to crash about reaction-admin

[zenweasel]

My .02. I think this is one of the last remnants of Meteor in admin and is probably a good candidate for a GQL query. This should be the standard pattern by now I would think

[loan-laux]

I'm taking on this one if you guys don't mind. Already started to implement the paginated query on api-plugin-email. PRs coming soon.

[nnnnat]

@loan-laux I got you. Someone will get these reviewed hopefully next week. Thank you for helping out here, very much appreciated.

[Akarshit]

@nnnnat I was looking into ways to solve this. This is what I have so far:
The query fetches all emails and is used by SortableTable, which expects all of the data to be present. This is used for filtering and pagination. If we were to change this, we would have to implement a few things like:

1. Pagination while fetching the query on the server.
2. Filtering/search on the server.
3. Modifying the SortabeTable to handle this new type of stream.

Wanted to get a confirmation if this seems like a good plan to go forward with. Another option is to make it a GQL query and do the same things as above.

[nnnnat]

Yeah I agree with @zenweasel here. We should try to move this to a GQL query and follow the same pagination/filtering patters we have for things like products and orders (hopefully minimize the amount of modifications to SortableTable). Probably not a small lift but the patterns should exist now @Akarshit if you want to take this on.

[loan-laux]

Thanks for assigning me, @nnnnat. Can you move this to In Progress on the Bugs & Stability Improvements board? Just to satisfy my PTSD and show some progress on that loading bar. 😄

[loan-laux]

@nnnnat, feel free to check out my two PRs when you have some time:

reactioncommerce/api-plugin-email#6
#350