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

Upgrade react admin to v4 #1436

Merged
merged 11 commits into from
Feb 3, 2024
Merged

Upgrade react admin to v4 #1436

merged 11 commits into from
Feb 3, 2024

Conversation

vcastellm
Copy link
Member

@vcastellm vcastellm commented Dec 9, 2023

https://marmelab.com/react-admin/ is used for the UI, this is the upgrade to v4, happy coincidence with Dkron V4.

This is a much necessary upgrade incorporating some breaking changes, mostly the UI works the same functionality wise, but the default color is changed.

The previous combination of dark grey and red was good looking but not very appropriate for UI, as the red color indicates errors or warning, going this time with the default material UI blue color.

@vcastellm vcastellm added dependencies Pull requests that update a dependency file 4.x labels Dec 9, 2023
@vcastellm vcastellm changed the title Upgrade react admin first shot Upgrade react admin to v4 Dec 10, 2023
@vcastellm vcastellm merged commit 7af72c3 into 4.x Feb 3, 2024
2 checks passed
@vcastellm vcastellm deleted the upgrade_react_admin branch February 3, 2024 17:53
vcastellm added a commit that referenced this pull request Oct 6, 2024
* Convert shell plugin to internal plugin (#975)

The purpose of this PR is to embed the shell plugin in the main dkron binary, that will facilitate creating a single binary with the most important executor, for easy deployment using a single binary.

* Remove old UI (#984)
* Release a light image tag (#988)

Omit all plugins except the shell plugin that will be included in the main binary.

* Upgrade react admin to v4 (#1436)

* Use exponential backoff for retries (#1433)

* Handle ip changes (#1446)
* consul like approach: add server_lookup to make dkron independent from server node IP on raft layer
* handle memberupdate event
* query other servers before start & add test
* don't remove itself if node is a raft leader
* don't remove dkron server node if id matches

* Move config init to agent command (#1465)

Config init was being done on the root level command but only the agent command was using config values.

Now config init is done as pre-run of agent command only, getting rid of extra messages in other commands when the config was missing.

* Fix disabled (#1467)
* Embed http plugin (#1471)

http plugin is one of the most used plugins together with the shell plugin, embedding it in the main binary allow for more lean deployment.

* Show all Job fields
* Refactor buttons for admin v4

* Docs v4 (#1473)

* Reuse http clients with the same configuration (#1474)
* Update OpenAPI spec to v3.1.0 and add ACLs spec
* Refactor location of types as they not only belong to plugins, but used in general. (#1485)

Also add Pro protobuf and gen code as this should be public.

* Generate client
* Extend protobuf defs for ACLs
* Token fields
* Minor improvement in RabbitMQ executor (#1500)
* nice and forkable goreleaser (#1584)
* add support for hash scheduling (#1260)
* feat: Login form and admin update (#1589)

Upgrade react-admin to v5
Implements a login form to use in Pro version

---------

Co-authored-by: Victor Castell <victor@victorcastell.com>
Co-authored-by: Ivan Kripakov <108407979+ivan-kripakov-m10@users.noreply.github.com>
Co-authored-by: Filipe Pina <636320+fopina@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant