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

Malware Quick Start #222

Merged
merged 3 commits into from
Aug 15, 2024
Merged

Conversation

CodeDuchess
Copy link
Contributor

quick start for Insights malware detection service

@app-sre-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@Hyperkid123 Hyperkid123 requested a review from a team August 6, 2024 08:42
@Hyperkid123
Copy link
Collaborator

/ok-to-test

@CodeDuchess
Copy link
Contributor Author

@Hyperkid123 How do I resolve the conflicts? I am thinking it probably needs rebasing, but I know I am not able to do that. Please advise. Thanks.

@Hyperkid123
Copy link
Collaborator

@RedHatInsights/platform-experience can you guys help @CodeDuchess?

@ryelo
Copy link
Member

ryelo commented Aug 14, 2024

@CodeDuchess Can you just reverify that everything looks correct?

@CodeDuchess
Copy link
Contributor Author

@ryelo It has had several peer reviews. So, it is good to go. Thanks.

@Hyperkid123 Hyperkid123 merged commit b1915e1 into RedHatInsights:main Aug 15, 2024
7 checks passed
@CodeDuchess
Copy link
Contributor Author

@Hyperkid123 @ryelo Thanks so much for your assistance with this PR! I am new to working with the console team, so I have a question. After a PR is merged, when does the new content show up in the UI? Just wanting to better understand the workflow. Thanks!

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.

4 participants