#50011 Issue 49999 - Finish up the transfer to React
Closed by spichugi. Opened by spichugi.
spichugi/389-ds-base react_trans_finish  into  master

Download 50011.patch

Description: Move plugins.es6 to index.es6 so we can
properly navigate using old nav-bar. New React entry points
should be added to the index.es6.
Add a make target for manual Cockpit UI building:
'make 389-console' and a target to node_modules.mk 'eslint-fix'.
Fix a few CSS issues.

https://pagure.io/389-ds-base/issue/49999

Reviewed by: ?

From this PR, you'll be able to fully join the UI development in React. :)
You can start to add your modules as 'secutiry.jsx', etc. and include it as it's done with 'plugins.jsx' Plugins.

I'm getting some console errors:

index.html:547 GET http://localhost.localdomain:9090/cockpit/@localhost/389-console/index.js net::ERR_ABORTED 404 (Not Found)
index.html:547 GET http://localhost.localdomain:9090/cockpit/@localhost/389-console/index.js net::ERR_ABORTED 404 (Not Found)

I'm getting some console errors:
index.html:547 GET http://localhost.localdomain:9090/cockpit/@localhost/389-console/index.js net::ERR_ABORTED 404 (Not Found)

index.html:547 GET http://localhost.localdomain:9090/cockpit/@localhost/389-console/index.js net::ERR_ABORTED 404 (Not Found)

Weird... Do you have the file? index.js
It should be generated after 'npm run build' (which is run in Makefile target)

How do you install it?
If you just do it locally you can run:

make 389-console-clean
make 389-console

There is no index.js I was just running it from my .local directory.

Also those commands do not work:

[mareynol@localhost 389-ds-base]$ make -f rpm.mk 389-console-clean
make: *** No rule to make target '389-console-clean'. Stop.

[mareynol@localhost 389-ds-base]$ make -f rpm.mk 389-console
make: *** No rule to make target '389-console'. Stop.

Either way LGTM - we can address any other issues later

There is no index.js I was just running it from my .local directory.
Also those commands do not work:
[mareynol@localhost 389-ds-base]$ make -f rpm.mk 389-console-clean
make: *** No rule to make target '389-console-clean'. Stop.
[mareynol@localhost 389-ds-base]$ make -f rpm.mk 389-console
make: *** No rule to make target '389-console'. Stop.

Okay.

The targets '389-console-clean', '389-console' and '389-console-devel-install' are in Makefile.am.

'make -f rpm.mk' is used for creating 'srpms' or 'dist-bz2' tarball (that are the places where cockpit is involved too)

So you need to run:

make 389-console-clean
make 389-console
make 389-console-devel-install  # if you haven't run it yet

rebased onto fb293b78cd7c9348c47fd9a2c451e819184ae29c

rebased onto ce05f24635d7f1aeaa6ac4fb20a2e5bb46553377

Pull-Request has been merged by spichugi

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3070

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata