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

Refactor/navigation and state mgmt #32

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

Conversation

vinnyA3
Copy link
Owner

@vinnyA3 vinnyA3 commented Jul 24, 2022

Description

Related Issues

Motivation and Context

How To Test It?

  • Environment: Linux
  • Node Version: v16.13.1
  • YT-DLP Version: 2021.12.27

Steps:

Visual reference

Checklist

  • I have tested on a supported environment: ENTER_ENV_HERE - MacOS, Linux
  • I have tested using the project's Node version: v16.13.1
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.

@vinnyA3 vinnyA3 marked this pull request as draft July 24, 2022 20:55
vinnyA3 added 11 commits October 1, 2022 21:44
In an effort to remove the UI logics away from the actual anime
processing, created a simple navigator object & screens to handle
rendering of different UI->prompts.  Additionally, paving way for
persisted state (history) & queue'd streaming+downloads, by
incorporating redux
Managing a custom redux implementation is going to be a bitch -- let's
not do that
* add redux & remove custom redux implementation
* update types
* add redux middleware for debug
@vinnyA3 vinnyA3 force-pushed the refactor/navigation-and-state-mgmt branch from e1f9f47 to 10b9867 Compare October 2, 2022 01:46
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.

1 participant