Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

React admin Web UI #864

Merged
merged 3 commits into from
Dec 17, 2020
Merged

React admin Web UI #864

merged 3 commits into from
Dec 17, 2020

Conversation

vcastellm
Copy link
Member

@vcastellm vcastellm commented Dec 10, 2020

Effort to rewrite the UI using https://marmelab.com/react-admin/

Just missing the Dashboard graph, no really useful TBH. Other features work better that before, also it's now much easier to implement improvements to the search, etc.

cc/ @yvanoers

@vcastellm vcastellm force-pushed the react_webui branch 2 times, most recently from 5e17b1b to 2752531 Compare December 17, 2020 22:00
Rewrite WebUI using React with Reac admin, using Typescript to match
project codebase, style and take advantage of its features.

Modification to the API are necessary to correctly use the features of
RA Datagrid, pager, etc.

It stores Jobs in JSON to BuntDB, but if existing values are present in
Proto format it converts them, this was necessary to use Sorting
capabilities using BundDB indexes.

The new UI is optional at the moment, it is possible to fallback to the
old ui by setting the --ui param to false
@vcastellm vcastellm changed the title [WIP] React webui React webui Dec 17, 2020
@vcastellm vcastellm changed the title React webui React admin Web UI Dec 17, 2020
@vcastellm vcastellm merged commit 995be97 into master Dec 17, 2020
@vcastellm vcastellm deleted the react_webui branch December 17, 2020 22:57
@fopina
Copy link
Contributor

fopina commented Dec 18, 2020

<3
Now I can't wait to find the time to try to rewrite to preact for smaller browser footprint!:P

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

Successfully merging this pull request may close these issues.

2 participants