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

Read pdf hash feature #32387

Merged
merged 28 commits into from Jan 25, 2024
Merged

Read pdf hash feature #32387

merged 28 commits into from Jan 25, 2024

Conversation

anas-yousef
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-32377

Description

Added support for extracting hashes.

Must have

  • Tests
  • Documentation

@anas-yousef anas-yousef self-assigned this Jan 23, 2024
@anas-yousef anas-yousef changed the title Ay read pdf hash feature Read pdf hash feature Jan 23, 2024
@anas-yousef
Copy link
Contributor Author

@JasBeilin We just need to decide how we want to update the documentation of the script.

Copy link
Contributor

@JasBeilin JasBeilin left a comment

Choose a reason for hiding this comment

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

Nice work, lets enhance a bit and merge :)

@anas-yousef anas-yousef marked this pull request as ready for review January 24, 2024 11:04
@anas-yousef anas-yousef merged commit e332fb2 into master Jan 25, 2024
16 checks passed
@anas-yousef anas-yousef deleted the ay-read-pdf-hash-feature branch January 25, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants