Skip to content
This repository has been archived by the owner on Jan 12, 2020. It is now read-only.

[#2]: filter by month slider #38

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jeff-s-zhou
Copy link

WIP, slider works but there's a minor bug with the default month displayed. Should also probably call the filter function on load by the current month

@Mokosha
Copy link
Contributor

Mokosha commented Jul 18, 2019

I took a look at this today. Looks like a good start.

We need to be able to fetch more data from the backend for a slider like this to be useful though. That's blocked on lanebreach/lanebreach-api#40 because the amount of data that we fetch ends up being uncompressed and taking way too much time to send. There's a bunch of better architectural changes that we need to implement between frontend and backend, but even sending compressed data will be enough to increase the latency here considerably.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants