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.

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

CoVariants v1 #375

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

CoVariants v1 #375

wants to merge 26 commits into from

Conversation

ivan-aksamentov
Copy link
Member

@ivan-aksamentov ivan-aksamentov commented Feb 15, 2023

CoVariants v0 is a pandemic-grade software and some of the technical decisions were rushed and questionable.

Let's start over? =D

List of changes:

  • Dynamic data fetching. Avoid storing the data in the GitHub repo. Let's move it to S3. This will require to change infallible, synchronous JSON imports (bundled into the app through webpack), into fallible and asynchronous network requests. Unfortunately these JSON imports are so entangled with the app components that it will be quite a task.
  • Fine-grained data fetching. Let's split the large JSONs into pieces and only fetch data that's being displayed.
  • Better user experience. There are numerous defects in the UI and UX to address. The list will follow.
  • ...

@vercel
Copy link

vercel bot commented Feb 15, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
covariants ❌ Failed (Inspect) Feb 15, 2023 at 2:32PM (UTC)

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

Successfully merging this pull request may close these issues.

None yet

1 participant