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

Settings page for the web UI #239

Merged
merged 60 commits into from
Dec 14, 2016
Merged

Settings page for the web UI #239

merged 60 commits into from
Dec 14, 2016

Conversation

DL6ER
Copy link
Member

@DL6ER DL6ER commented Dec 11, 2016

Changes proposed in this pull request:

  • Add a settings page to the web UI

To be able to use it you will have to check out the branch websettingspage for pi-hole core. In the top row you see some debug output. I will remove this at some later stage.

Screenshot:
screenshot at 2016-12-11 19-51-26

Does not work without pi-hole/pi-hole#976

@pi-hole/dashboard

@C4Wiz
Copy link

C4Wiz commented Dec 11, 2016

the toggles for the blocking page do not work, nor do the admin buttons

@dschaper
Copy link
Member

WIP = Work In Progress

@DL6ER
Copy link
Member Author

DL6ER commented Dec 11, 2016

@C4Wiz The options where I haven't implemented the backend yet are marked as disabled simply because nothing would happen

@C4Wiz
Copy link

C4Wiz commented Dec 11, 2016

yea, sorry, i jumped the gun....my bad

@DL6ER
Copy link
Member Author

DL6ER commented Dec 12, 2016

@C4Wiz Try again 😉

@PromoFaux
Copy link
Member

Pefect.

I'm unable to test the DHCP settings currently. Someone else may want to check that before approval

@PromoFaux
Copy link
Member

PromoFaux commented Dec 14, 2016

Approved

Approved with PullApprove

@PromoFaux PromoFaux merged commit 8e6cb32 into devel Dec 14, 2016
@DL6ER DL6ER deleted the settings branch December 14, 2016 19:12
@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

unticking the boxed layout option and clicking save changes the layout, but when navigating to the main page the layout returns to boxed!

screen shot 2016-12-20 at 9 09 54 am
screen shot 2016-12-20 at 9 10 12 am

@PromoFaux
Copy link
Member

@C4Wiz, this may have been fixed in another branch. Are you on the devel branch?

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

yes i am

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

Can you tell me what version is in your footer?

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

screen shot 2016-12-20 at 9 13 54 am

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

What happens if you visit the Settings page, again? Is the box selected although you deactivated it before?

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

yes

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

Unfortunately, I cannot confirm this on the very same git versions as you are.

Please try the following:

  • Change it on the web UI, does WEBUIBOXEDLAYOUT= in /etc/pihole/setupVars.conf change?
  • Try switching if off using: sudo pihole -a layout traditional, does /etc/pihole/setupVars.conf change?

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

there is no WEBUIBOXEDLAYOUT= in the setupVars.conf

that command does nothing

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

Then your scripts seem outdated. You have to run pihole -r after pulling changes from the development branch of the core repository.

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

oh sorry, i was not aware the pihole -r was required, running now

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

Yes, they are installed on an update, while installing means copying with proper permissions. You just updated the scripts in the local repository but not the ones at the installation targets /usr/local/bin/ and /opt/pihole

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

gotcha, kinda new to git...confirmed it is now working properly. and i learned something new!

@DL6ER
Copy link
Member Author

DL6ER commented Dec 20, 2016

Every one of us has benefited. You learned something new and we know that it works for you as well.

@C4Wiz
Copy link

C4Wiz commented Dec 20, 2016

you guys are awsome!

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.

5 participants