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

theharvester: add Spanish translation #12769

Merged
merged 6 commits into from May 13, 2024
Merged

theharvester: add Spanish translation #12769

merged 6 commits into from May 13, 2024

Conversation

kant
Copy link
Member

@kant kant commented May 9, 2024

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@kant kant added the translation Translate pages from one language to another. label May 9, 2024
tricantivu
tricantivu reviewed May 12, 2024
View reviewed changes
Copy link
Member

@tricantivu tricantivu left a comment

Choose a reason for hiding this comment

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

Is the command name really TheHarvester?

pages.es/common/theharvester.md Outdated Show resolved Hide resolved
pages.es/common/theharvester.md Outdated Show resolved Hide resolved
pages.es/common/theharvester.md Outdated Show resolved Hide resolved
pages.es/common/theharvester.md Outdated Show resolved Hide resolved
pages.es/common/theharvester.md Outdated Show resolved Hide resolved
@sebastiaanspeck
Copy link
Member

Is the command name really TheHarvester?

Yep. https://github.com/laramies/theHarvester

@tricantivu
Copy link
Member

tricantivu commented May 12, 2024 •
edited

Is the command name really TheHarvester?

Yep. https://github.com/laramies/theHarvester

Huh, I guess I was accustomed to lowercase command names. Maybe this source file confirms it

kant and others added 5 commits May 12, 2024 19:47
@kant @tricantivu
Update pages.es/common/theharvester.md
4a6649a 
Co-authored-by: Alejandro Cervera <96702705+tricantivu@users.noreply.github.com>
@kant @tricantivu
Update pages.es/common/theharvester.md
0927732 
Co-authored-by: Alejandro Cervera <96702705+tricantivu@users.noreply.github.com>
@kant @tricantivu
Update pages.es/common/theharvester.md
bde8a52 
Co-authored-by: Alejandro Cervera <96702705+tricantivu@users.noreply.github.com>
@kant @tricantivu
Update pages.es/common/theharvester.md
382a3f1 
Co-authored-by: Alejandro Cervera <96702705+tricantivu@users.noreply.github.com>
@kant @tricantivu
Update pages.es/common/theharvester.md
ec0c2c3 
Co-authored-by: Alejandro Cervera <96702705+tricantivu@users.noreply.github.com>
@tricantivu
Copy link
Member

I just thought of something, shouldn't the page be in common? AFAIK, it meets the requirements.

@sebastiaanspeck
Copy link
Member

sebastiaanspeck commented May 13, 2024 •
edited

I just thought of something, shouldn't the page be in common? AFAIK, it meets the requirements.

You are correct, since it is a Python-based tool, but it is already in common 😄

@sebastiaanspeck sebastiaanspeck merged commit 3545dc9 into main May 13, 2024
7 checks passed
@sebastiaanspeck sebastiaanspeck deleted the kant-patch-3 branch May 13, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebastiaanspeck sebastiaanspeck sebastiaanspeck approved these changes

@tricantivu tricantivu tricantivu approved these changes

@navarroaxel navarroaxel Awaiting requested review from navarroaxel navarroaxel is a code owner

Assignees
No one assigned
Labels
translation Translate pages from one language to another.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kant @sebastiaanspeck @tricantivu