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

Cbi settings #26

Merged
merged 14 commits into from
Mar 10, 2014
Merged

Cbi settings #26

merged 14 commits into from
Mar 10, 2014

Conversation

seamustuohy
Copy link
Contributor

This commit will restructure the way that nodogsplash is handled in the luci menu's.

SEE: opentechinstitute/luci-commotion#195
SEE: opentechinstitute/commotion-feed#50

To Test:

  1. Build and image from the following commotion-feed branch: "commotion-splash-cbi"
  2. Add the following branches for makefiles to pull from:
    luci-commotion-splash: cbi_settings
    luci-commotion: commotion-splash-settings
  3. Complete the setup wizard.
  4. Enable the welcome page in "Basic --> Client Controls"
  5. Confirm that the welcome page was added.
  6. Go to "Advanced --> Services --> Captive Portal"
  7. Perform a variety of nodogsplash configurations on this page.
  8. Confirm that the nodogsplash config file is correct
  9. When nodogsplash is supposed to be on confirm that ndsctl status affirms the correct setttings
  10. when off, ndsctl status should show it as off.
  11. All settings should be reflected in the splash page options

Andrew Reynolds and others added 6 commits March 4, 2014 14:40
…nto /etc/init.d/ucidog, uci2dns.lua, and nds_config.lua so that we could make the interface cbi compatable.
This is one of the many inconsistancies between these formats
@jheretic jheretic added this to the 1.1 milestone Mar 7, 2014
@jheretic jheretic self-assigned this Mar 7, 2014
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.

2 participants