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

PDFToolBox #216

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

PDFToolBox #216

wants to merge 8 commits into from
+128 −17

Conversation

shrehs
Copy link
Contributor

@shrehs shrehs commented Oct 31, 2024

Closes: #215

  • Title: Completion of PDFToolbox
  • Your Name: Shreya H S
  • Idenitfy yourself: GSSOC-Ext, Hacktoberfest

Describe the add-ons or changes you've made 📃

Implement mergePDF, splitPDF, rotatePDF, encryptPDF, decryptPDF functions

Give a clear description of what have you added or modifications made

Checklist: ☑️

  • My code follows the Contributing Guidelines & Code of Conduct of this project.
  • This PR does not contain plagiarized content.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly wherever it was hard to understand.
  • My changes generate no new warnings.

Screenshots 📷

PDFToolbox1.webm
Screenshot (144)
Screenshot (145)

@github-actions GitHub Actions
Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@Zedoman
Copy link
Collaborator

Zedoman commented Oct 31, 2024 •
edited
Loading

Also provide a video for this @shrehs

@Zedoman Zedoman self-requested a review October 31, 2024 17:52
@shrehs
Copy link
Contributor Author

shrehs commented Oct 31, 2024

Also provide a video for this @shrehs

I have! It is just above the image (underlined blue font)

@Zedoman Zedoman self-assigned this Oct 31, 2024
Zedoman
Zedoman previously approved these changes Oct 31, 2024
View reviewed changes
@Zedoman Zedoman removed their assignment Oct 31, 2024
@Zedoman Zedoman added enhancement 🧑‍💻 New feature or request gssoc-ext gssoc extended 2024 hacktoberfest-accepted Hacktoberfest 2024 labels Oct 31, 2024
Repository owner deleted a comment from github-actions bot Oct 31, 2024
Avdhesh-Varshney
Avdhesh-Varshney requested changes Nov 1, 2024
View reviewed changes
Copy link
Owner

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

Remove these 2 files from your commit

src/apps/pages/programs/ApiPrograms/recipe_finder/.gitignore Outdated Show resolved Hide resolved
src/apps/pages/programs/ApiPrograms/recipe_finder/recipe_finder.py Outdated Show resolved Hide resolved
@Avdhesh-Varshney Avdhesh-Varshney added bug 🐛 Something isn't working Requested Changes ⚙️ Some changes have been requested in this PR labels Nov 1, 2024
@shrehs
Copy link
Contributor Author

shrehs commented Nov 1, 2024

Remove these 2 files from your commit

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Avdhesh-Varshney Avdhesh-Varshney Avdhesh-Varshney requested changes

@Zedoman Zedoman Zedoman left review comments

Requested changes must be addressed to merge this pull request.

Assignees

@shrehs shrehs

Labels
bug 🐛 Something isn't working enhancement 🧑‍💻 New feature or request gssoc-ext gssoc extended 2024 hacktoberfest-accepted Hacktoberfest 2024 Requested Changes ⚙️ Some changes have been requested in this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🧑‍💻: Completing the Todo in PDFToolbox
3 participants
@shrehs @Zedoman @Avdhesh-Varshney