Skip to content

Configure nameservers#271

Merged
teclator merged 2 commits intofeature/network-overviewfrom
write_nameservers_config
Oct 20, 2022
Merged

Configure nameservers#271
teclator merged 2 commits intofeature/network-overviewfrom
write_nameservers_config

Conversation

@teclator
Copy link
Contributor

Problem

Currently the ipv4 dns configuration is not touched, we should allow to set some DNS server specially when the IP configuration is statically configured.

Solution

Modify ipv4 dns configuration when the settings are touched.

Testing

  • Tested manually

Screenshots

Screenshot from 2022-10-20 13-24-20

@teclator teclator force-pushed the write_nameservers_config branch from d8533c1 to 181a3ed Compare October 20, 2022 12:56
@teclator teclator force-pushed the write_nameservers_config branch from 181a3ed to e081d2b Compare October 20, 2022 13:16
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3289997799

  • -19 of 27 (29.63%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.09%) to 73.023%

Changes Missing Coverage Covered Lines Changed/Added Lines %
web/src/IpSettingsForm.jsx 0 4 0.0%
web/src/utils.js 4 19 21.05%
Totals Coverage Status
Change from base Build 3282084053: 1.09%
Covered Lines: 1708
Relevant Lines: 2339

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3289997799

  • 8 of 27 (29.63%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.09%) to 73.023%

Changes Missing Coverage Covered Lines Changed/Added Lines %
web/src/IpSettingsForm.jsx 0 4 0.0%
web/src/utils.js 4 19 21.05%
Totals Coverage Status
Change from base Build 3282084053: 1.09%
Covered Lines: 1708
Relevant Lines: 2339

💛 - Coveralls

@teclator teclator merged commit 87227a8 into feature/network-overview Oct 20, 2022
@teclator teclator deleted the write_nameservers_config branch October 20, 2022 13:38
@imobachgs imobachgs mentioned this pull request Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants