Skip to content

Modify Claim Screen (Windows Installer)#19132

Merged
thiagoftsm merged 2 commits intonetdata:masterfrom
thiagoftsm:hide_options
Dec 6, 2024
Merged

Modify Claim Screen (Windows Installer)#19132
thiagoftsm merged 2 commits intonetdata:masterfrom
thiagoftsm:hide_options

Conversation

@thiagoftsm
Copy link
Contributor

@thiagoftsm thiagoftsm commented Dec 5, 2024

Summary

This PR is modifying our installer to hide all fields on Claim Screen when there is already a claim.conf file generated during installation time.

Test Plan
  1. Compile this branch
  2. Make an installer
  3. Remove your netdata using Windows Control Panel
  4. If you already claimed your agent, rename your c:\Program Files\Netdata\etc\netdata\claim.conf and run the installer. You should have the following screen:

without

  1. Stop the installation, and rename the file again to c:\Program Files\Netdata\etc\netdata\claim.conf . Run the installer again, this time you will have the following screen:

with

Additional Information
For users: How does this change affect me? Describe the PR affects users: - Which area of Netdata is affected by the change? Installer - Can they see the change or is it an under the hood? If they can see it, where? Only when installing software. - How is the user impacted by the change? They have a clear vision about connection with cloud. - What are there any benefits of the change? Avoid confusions and save time during installation.

@thiagoftsm thiagoftsm requested a review from stelfrag December 5, 2024 16:32
@thiagoftsm thiagoftsm requested a review from a team as a code owner December 5, 2024 16:32
@github-actions github-actions bot added the area/packaging Packaging and operating systems support label Dec 5, 2024
@thiagoftsm thiagoftsm merged commit e4f9a5a into netdata:master Dec 6, 2024
@thiagoftsm thiagoftsm deleted the hide_options branch December 6, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/packaging Packaging and operating systems support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants