-
Notifications
You must be signed in to change notification settings - Fork 431
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
Update A1 - Copy-n-Paste README #165
Conversation
@rafaeleyng, @vitoriario and @Krlier , would you mind reviewing this one, please? I am setting this as WIP so we can work on this together! |
|
||
<img src="images/CopyNPaste.png" align="center"/> | ||
<img src="../../../images/secDevLabs-logo.png" align="" height="80" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this first image should be about the app! Not necessarily the app's home page, but maybe it's logo? It would be cool if we could add visual identity to each one of our apps, just like Hack The Box does with it's boxes. What do you guys think @vitoriario , @rafaeleyng ?
It doesn't need to be anything fancy, hack the box has a template and just changes some details to each box to kind of fit the theme. Here's an example:
OR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using logos is a good idea, however it may take us some time to build "cool" logos for each app. I may need some help doing so! 😃
Also, we have to be careful when using "Difficult: Hard". How will we define what is Hard
or what is Easy
? Taking vulnhub.com apps as examples, I always had some trouble working on "Beginners" machines and went pretty well on "Medium/Hard" ones. The challenge may be easy for the author but not to whom does it.
Hi guys, I think we could move all attack pictures from docs to images folder. What do you think? |
@vitoriario , thanks for this new commit! 👏🏻 However, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good to me. I think the logo would add some complexity in creating new apps.
I'm still missing a topic that states clearly the expected results after mitigation.
[FEAT] Redirect / to /login
- Fix indicated file (data is contained in `config.yml` instead of `docker-compose.yml`); - Add address and port information for connecting to Robo3T in README.
Link to `contact.php` file was broken.
[DOCS] Fix Objectives link A2
[DOCS] Fix Objectives link A3
[DOCS] Fix Objectives link A2
[DOCS] Fix Objectives link A4
[DOCS] Fix Objectives link A5
[DOCS] Update A3 README
[DOCS] Fix broken link in A4 README
This PR is a draft to solve #163 .