search

human-tools / toolbox-web

Simple Secure and Private Tools for everyday humans
https://humantools.io
21 stars 3 forks source link

Add delete feature to combine pdf tool #14

Closed Fahme closed 1 year ago

Fahme commented 3 years ago

Adds a delete feature to the merge pdf tool

Fixes on of the issues in https://github.com/human-tools/toolbox-web/issues/10

mkhatib commented 3 years ago

btw, if you'd like we could jump on a video call and pair program on this (something @ahmgeek and I have been doing). Usually we do it on Sundays between 9am-1pm PST.

Fahme commented 3 years ago

@mkhatib thanks for the invite but it will be quite late for me to work at that time. I'm going to tackle this PR during the weekends.

Feel free to submit a solution if you guys worked on this before.

mkhatib commented 3 years ago

Nah take your time. There are bunch other stuff to work on.

On Fri, Apr 16, 2021, 9:58 AM Fahmi Bnchi @.***> wrote:

@mkhatib https://github.com/mkhatib thanks for the invite, but it will be quite late for me to work at that time, i'm going to tackle this PR during the weekends, but feel free to submit a solution if you guys worked on this before.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/human-tools/toolbox-web/pull/14#issuecomment-821310855, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAAUVYFEYJQXPJVJCFX6ALTJBUEDANCNFSM42YFPWEQ .

mkhatib commented 1 year ago

Thanks @Fahme for working on this. I just added the feature to delete pages in combine PDF tools. Closing this.