Skip to content

[WIP] React Web UI#1854

Merged
liiight merged 66 commits intodevelopfrom
react-ui
Oct 18, 2017
Merged

[WIP] React Web UI#1854
liiight merged 66 commits intodevelopfrom
react-ui

Conversation

@kristenmills
Copy link
Copy Markdown
Member

@kristenmills kristenmills commented May 31, 2017

Starting a WIP pull request for better visibility.

Motivation for changes

Nobody wants to write Angular 1. Now they don't have to! 💯

Detailed changes:

  • Rewrite existing web ui using modern technologies (ES2015+, React, Redux, Webpack)

To Do:

https://github.com/Flexget/Flexget/projects/4

Wiki

https://flexget.com/Web-UI/v2

@liiight
Copy link
Copy Markdown
Member

liiight commented May 31, 2017

Well I already see what you meant by react being more understandable than angular...
Not sure I'd name the packages old and new ui. V1 and v2 fits better imo, we'll probably run them side by side for a long while until we nail down v2

@kristenmills
Copy link
Copy Markdown
Member Author

Yeah that makes sense. I'll make the change.

dev_tools.py Outdated
# Install bower packages
click.echo('running `bower install`')
subprocess.check_call('bower install -p', cwd=cwd, shell=True)
subprocess.check_call('bower install', cwd=cwd, shell=True)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kristenmills Maybe we should switch to Yarn? bower/bower#2298

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bower was used on the old ui not the new one. We're using npm 5 on the new one.

@stevezau
Copy link
Copy Markdown
Contributor

stevezau commented Oct 4, 2017

Hey @kristenmills i'm having a look at this. Can you help me understand the layout of the folders etc? Also the snapshots folders?

@kristenmills
Copy link
Copy Markdown
Member Author

I'd like to try to merge this and then start doing smaller, feature based pull requests. Anyone opposed?
cc: @liiight @cvium @stevezau

Also does someone mind reviewing the python portions to make sure i'm not doing anything ridiculous?

@liiight
Copy link
Copy Markdown
Member

liiight commented Oct 15, 2017

Sounds good to me

from . import __path__

__version__ = '1.3.0'
__version__ = '1.4.0'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For non breaking api changes we usually just bump patch. No biggie though

'ssl_private_key': {'type': 'string'},
'web_ui': {'type': 'boolean'},
'base_url': {'type': 'string'},
'version': {'type': 'integer',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that it'd be better to have an ability to run both uis simultaneously

@stevezau
Copy link
Copy Markdown
Contributor

yeah merge it in i'd say.. assuming v1 still works fine?

I've been learning reactjs-redux and am already ready to start work on this also.

@liiight
Copy link
Copy Markdown
Member

liiight commented Oct 15, 2017

I think that until v1 can be completely deprecated, v2 should be able to run along side it and not replace it

@stevezau
Copy link
Copy Markdown
Contributor

v2 is not ready to take the place of v1 just yet. Just saying we need to keep v1 as default for now.

@kristenmills
Copy link
Copy Markdown
Member Author

Yeah it's set up so that v1 is the default. I'm working right now to configure v2 to run along aside.

@kristenmills
Copy link
Copy Markdown
Member Author

Now the v1 ui runs at root. if you add run_v2: true, v2 will run at /v2. Sound good?

@liiight
Copy link
Copy Markdown
Member

liiight commented Oct 16, 2017

Good enough for now but ideally I think having a list key will be nicer:

web_ui:
- v1
- v2

@stevezau
Copy link
Copy Markdown
Contributor

agree, run side by side..

@kristenmills
Copy link
Copy Markdown
Member Author

Yeah key listing is probably better. I'll do that after work today.

@liiight liiight merged commit fb622fb into develop Oct 18, 2017
@gazpachoking gazpachoking deleted the react-ui branch January 28, 2019 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants