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

Remove "not available" message #10998

Merged
merged 2 commits into from
Aug 21, 2023

Conversation

jan-cerny
Copy link
Collaborator

In the HTML controls pages, the variables are shown with a "not available in this product" message, which is incorrect. This message makes sense only with rules selections but not with variables selections.

Review Hints:

python3 utils/render_all_policies.py --output-dir /tmp/out --ssg-root $(pwd) --product rhel9
firefox /tmp/out/anssi.html

In the HTML controls pages, the variables are shown with
a "not available in this product" message, which is incorrect.
This message makes sense only with rules selections but not with
variables selections.
@jan-cerny jan-cerny added this to the 0.1.70 milestone Aug 16, 2023
@github-actions
Copy link

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@marcusburghardt marcusburghardt self-assigned this Aug 17, 2023
@marcusburghardt marcusburghardt added the Documentation Update in project documentation. label Aug 17, 2023
The readers of the policy HTML page will be able to click
on a link that leads to the upstream definition of an XCCDF Value
on GitHub. This is similar to the link that we provide for rules.
@jan-cerny
Copy link
Collaborator Author

I have add a HTML link to XCCDF Values

@codeclimate
Copy link

codeclimate bot commented Aug 21, 2023

Code Climate has analyzed commit 58e3355 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 53.3% (0.0% change).

View more on Code Climate.

Copy link
Member

@marcusburghardt marcusburghardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@marcusburghardt marcusburghardt merged commit 2bf1870 into ComplianceAsCode:master Aug 21, 2023
32 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Update in project documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants