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.

Sign up for GitHub

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

Jump to bottom

Added Tryhackme Site #6188

Merged
merged 13 commits into from
Oct 12, 2021
Merged

Added Tryhackme Site #6188

merged 13 commits into from
Oct 12, 2021

Conversation

whiskerwind
Copy link
Contributor

No description provided.

@phallobst phallobst added the add site Issue/PR adds a site to the repo. label Oct 11, 2021
ApCoder123
ApCoder123 requested changes Oct 11, 2021
View reviewed changes
entries/t/tryhackme.com.json Outdated Show resolved Hide resolved
whiskerwind and others added 4 commits October 12, 2021 00:52
@whiskerwind @ApCoder123
Fix Domain Name
2b6c91e 
Co-authored-by: Arun Pattni <20560218+ApCoder123@users.noreply.github.com>
@ApCoder123
Add img tag
0eab31e
@ApCoder123
Fix JSON
777464b
@ApCoder123
Fix img path
cb145ae
@phallobst
Copy link
Member

How about this SVG icon:

@whiskerwind
Copy link
Contributor Author

Good idea on the logomark instead of the logotype

@whiskerwind @phallobst
Update categories
3f4ebd8 
Co-authored-by: Michael Loßin <github@michael-lossin.de>
@whiskerwind whiskerwind requested a review from phallobst October 12, 2021 06:30
@ApCoder123 ApCoder123 merged commit 69e65d3 into 2factorauth:master Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ApCoder123 ApCoder123 ApCoder123 approved these changes

@phallobst phallobst phallobst approved these changes

Assignees
No one assigned
Labels
add site Issue/PR adds a site to the repo.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@whiskerwind @phallobst @ApCoder123