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

HTML formatting and update doc link #16

Merged
merged 2 commits into from
Mar 31, 2023
Merged

Conversation

CodeByRRR
Copy link
Contributor

@CodeByRRR CodeByRRR commented Mar 30, 2023

Remove Protect AI logo from header. We will still have the Powered by: Protect AI text at the bottom of the report.
Update the docs link to the new url: https://nbdefense.ai/
Remove Protect AI from the title

image

@CodeByRRR CodeByRRR requested a review from milesjos March 30, 2023 19:29
milesjos
milesjos previously approved these changes Mar 30, 2023
Copy link
Member

@milesjos milesjos left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢

cherbel
cherbel previously approved these changes Mar 30, 2023
Copy link
Member

@cherbel cherbel left a comment

Choose a reason for hiding this comment

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

LGTM as well. It looks like you'll have to set up commit verification before this is merged.

@CodeByRRR CodeByRRR dismissed stale reviews from cherbel and milesjos via 9623c22 March 31, 2023 17:46
@CodeByRRR CodeByRRR force-pushed the mo_nbdefense_html_change branch from cb01102 to 9623c22 Compare March 31, 2023 17:46
@CodeByRRR CodeByRRR force-pushed the mo_nbdefense_html_change branch from 9623c22 to e493d55 Compare March 31, 2023 18:35
Copy link
Member

@milesjos milesjos left a comment

Choose a reason for hiding this comment

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

Code LGTM and the commits are verified!

@CodeByRRR CodeByRRR merged commit 1e1fe6e into main Mar 31, 2023
@CodeByRRR CodeByRRR deleted the mo_nbdefense_html_change branch March 31, 2023 18:58
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