Comments (16)
Hi @feckert, I've just commited many more changes. (you can take a look at PR #35)
It includes a complete rework of the menu and navigation.
It would be great if you can help testing it.
from luci-ng.
Hi @Lanchi thanks for reply. I am now testing it, and try to understand your changes :-).
The first lock and feel is great. Go on
from luci-ng.
Hallo @ianchi I tested your changes. But actually the "network/route" and the other uci/cbi pages does not working anymore as expected.
Failure:
The routing rules will not display and i also can not add new rules only the 'config route' in '/etc/config/network' will be set.
Questions:
Does this have to do with angular "matrial"?
Are there further changes planed in cbi?
from luci-ng.
Certainly many changes are pending to cbi but in the meantime this should still work as before.
I'll take a look at it as soon as I can.
I appreciate your reports and help testing.
from luci-ng.
@feckert, a question, does it simple don't save changes or do you get into a login loop?
from luci-ng.
After some digging I think there are (at least) two problems that go back to this commit 7487b57 when I splited cbi directives in code + template and started using transclusion.
- one problem is a syntax error on the template on cbiNetworkList, which I'll fix asap. This was causing the control not to render
- there is a deeper problem, introduced with the switch to transclusion. Now scopes don't inherit in the same way cbiOption supposes, and so sections can't be saved. This is more complex to solve, and as cbi is undergoing a rebumping, I don't know if it is worth the effort. I'll see if I can figure out some work arround for the meantime.
from luci-ng.
@Lanchi If I press save, then the page will gray out and nothing happens.
Could this have todo with this line?
https://github.com/jow-/luci-ng/blob/master/luci2-ui-base/src/www/luci-ng/cbi/cbiMap.js#L62
The function "self.read" is not defined?
from luci-ng.
@feckert, I've just pushed a fix for the above problems:
- transclusion/scopes problems that were preventing to save changes: temporary revert the use of templates+transclusion, till a deeper rework of cbi is done
- readded cbiMap.read() to complete the save path, and remove the "disabled" state, which left the page grayed out
- fix syntax error in NetworkList, which was preventing it from displaying.
I'd appreciate if you can do some more thorough tests, to be sure all cases are covered.
As you can see, we are currently in the middle of some heavy changes (many with side effects), and all your help in testing is very valuable.
By the way, as all these commits only involve client code, you don't need to continuously flash the router for testing. You can pull updates to your git repo, and then go to luci2-ui-base/src
and run gulp serve
(first you need to prepare the instance, as explained in BUILD.md, in general this step is needed only once or if there are any new dependencies).
This will run a local server that forwards all ubus request to your router. Then you point your browser to localhost:3000
and you'll get luci-ng as if you where on the router.
The port may vary if it is taken (it will be shown when running gulp). The address of your router must first be set in src/gulp/conf.js
.
from luci-ng.
@ianchi I am working with x86_64 and lantiq_xrx200 since you have added the line "PKG_BUILD_DEPENDS:=node/host" i have problem to compile node for the host system on target lantiq_xrx200(mips24k). The Target x86_64 build is working as expected.
On Target lantiq_xrx200(mips24k) I always get the following errors during build
g++ '-DGTEST_DONT_DEFINE_ASSERT_EQ=1' '-DGTEST_DONT_DEFINE_ASSERT_GE=1' '-DGTEST_DONT_DEFINE_ASSERT_GT=1' '-DGTEST_DONT_DEFINE_ASSERT_LE=1' '-DGTEST_DONT_DEFINE_ASSERT_LT=1' '-DGTEST_DONT_DEFINE_ASSERT_NE=1' -I../src -I../deps/v8/include -I../deps/gtest/include -pthread -Wall -Wextra -Wno-unused-parameter -m64 -O3 -ffunction-sections -fdata-sections -fno-omit-frame-pointer -fno-rtti -fno-exceptions -std=gnu++0x -MMD -MF /home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out/Release/.deps//home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out/Release/obj.target/cctest/test/cctest/util.o.d.raw -I/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/staging_dir/host/include -I/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/staging_dir/host/usr/include -I/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/staging_dir/target-mips_24kc_musl-1.1.16/host/include -c -o /home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out/Release/obj.target/cctest/test/cctest/util.o ../test/cctest/util.cc
make[6]: execvp: printf: Argument list too long
deps/openssl/openssl.target.mk:862: recipe for target '/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out/Release/obj.target/deps/openssl/libopenssl.a' failed
make[6]: *** [/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out/Release/obj.target/deps/openssl/libopenssl.a] Error 127
make[6]: *** Waiting for unfinished jobs....
make[6]: Leaving directory '/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/out'
Makefile:45: recipe for target 'node' failed
make[5]: *** [node] Error 2
make[5]: Leaving directory '/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5'
Makefile:93: recipe for target '/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/.built' failed
make[4]: *** [/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/lede/build_dir/target-mips_24kc_musl-1.1.16/host/node-v4.4.5/.built] Error 2
make[4]: Leaving directory '/home/bbslave/buildSlave/lede_dev_lantiq_xrx200/System6/build/feeds/packages/lang/node'
I think the important line is
make[6]: execvp: printf: Argument list too long
I already opened a ticket on lede with this issue
https://forum.lede-project.org/t/build-failure-for-node-v4-4-5-on-lede-17-01/1566
from luci-ng.
It is strange as the host is the same for all targets.
This is something related to LEDE's build system, perhaps @jow- can help us here.
If you already have node globally installed in your host, you could remove the host dependency altogether.
It is added to be sure that the proper tools are present in the build environmet. Perhaps we could automate this and only add the dependency if nodejs is not already globally installed in the system.
from luci-ng.
@ianchi i have solved my issue with node.
But i had to patch the sources witch small changes with this patch nodejs/node#9141 from @NeoRaider.
from luci-ng.
Great!
Was the original problem with saving on LUCI-NG solved after the fix I pushed?
from luci-ng.
@ianchi thanks for the fast fix :-) it is working again
from luci-ng.
Backup functionality is not working anymore.
I always get an "backup.tar.gz" with 33Bytes.
The same for Upgrade i could not upload a file.
Output "Web Console"
Unknown provider: $modalProvider <- $modal <- SystemSoftwareController <-
from luci-ng.
Ups!
The upgraded Angular Boostrap version changed module prefixes (now it is $uibModal instead of $modal). Some time back I changed all references in the "core" files, but obviouslly left some of the "views" controllers out...
Will complete the change asap (this is temporary, as in the middle run we plan to drop Boostrap library).
Again, thanks for your testing, it is very much appreciated (and needed).
from luci-ng.
Hi @feckert, I've pushed a fix. Pleas confirm that it is working.
from luci-ng.
Related Issues (20)
- IRC Minutes
- luci-ng blocks build HOT 2
- How to use the new node.js tools? HOT 1
- Why not continue to develop?
- (feat) Switch to websockets based transport for Ubus HOT 1
- (feat) Integrate legacy Luci views inside new Luci-ng app HOT 1
- What is the follow-up development plan for luci-ng? HOT 1
- (feat) ui-core: save workflow
- luci-vue HOT 1
- luci2-ui-base-2017-11-27-e439ce6 doesn't build HOT 1
- luci2-ui-base-2018-06-12 build failed on Ubuntu 18.04, Thanks HOT 1
- can ui-core be run on windows or only on openwrt? HOT 1
- What is a status of the project? HOT 3
- Add a license to the project. HOT 2
- Issues with local time
- Consider using Mithril+Tachyons for UI instead of Angular
- it's still developed ?
- luci2 build error openwrt 19.07.3 HOT 1
- Build Failed on Openwrt 19.07.7 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from luci-ng.