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

Add checks catalog view #241

Merged
merged 8 commits into from
Mar 25, 2022
Merged

Add checks catalog view #241

merged 8 commits into from
Mar 25, 2022

Conversation

arbulu89
Copy link
Contributor

Checks catalog view.

I have changed the catalog json struct. Have a look please. If you like it, I would change the runner to create that struct.
Now, the checks catalog can be updated based on the provider. The data is loaded at initialization, so we wouldn't have real updates if the catalog is changed in the runner.

I didn't update the package-lock.json file, as I think I have a different npm version, and it changes the whole file...*
This is just a frontend work. It doesn't do anything in the backend. We will discuss about this afterwards.
catalog

@arbulu89 arbulu89 force-pushed the add-checks-catalog-view branch from 5bd7a4f to 4c3495a Compare March 25, 2022 10:50
@stefanotorresi
Copy link
Member

stefanotorresi commented Mar 25, 2022

@arbulu89 would it be possible to preserve the code blocks around the parameters in the description, like it was before?

@arbulu89
Copy link
Contributor Author

arbulu89 commented Mar 25, 2022

@arbulu89 would it be possible to preserve the code blocks around the parameters in the description, like it was before?

Do you mean in the main description text? Yes, of course

Done in: 5bdd07f

image

@arbulu89 arbulu89 force-pushed the add-checks-catalog-view branch from 74c4243 to 931ca4d Compare March 25, 2022 13:37
Copy link
Contributor

@dottorblaster dottorblaster left a comment

Choose a reason for hiding this comment

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

LGTM, if you want to address the tiny comment I made... otherwise it's alright :D

assets/js/components/ChecksCatalog/ChecksCatalog.jsx Outdated Show resolved Hide resolved
@arbulu89 arbulu89 merged commit c3c2ab5 into main Mar 25, 2022
@arbulu89 arbulu89 deleted the add-checks-catalog-view branch March 25, 2022 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

4 participants