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

Adding Pairdrop to File Transfer - Single-click & Drag-n-drop Upload #3821

Merged
merged 3 commits into from May 25, 2023

Conversation

kgleeson
Copy link
Contributor

@kgleeson kgleeson commented May 11, 2023

Adding Pairdrop

Thank you for taking the time to work on a PR for Awesome-Selfhosted!

To ensure your PR is dealt with swiftly please check the following:

  • Submit one item per pull request. This eases reviewing and speeds up inclusion.
  • Format your submission as follows, where Demo and Clients are optional.
    Do not add a duplicate Source code link if it is the same as the main link.
    Keep the short description under 250 characters and use sentence case
    for it, even if the project's webpage or readme uses another capitalisation
    such as title case, all caps, small caps or all lowercase.
    Demo links should only be used for interactive demos, i.e. not video demonstrations.
    - [Name](http://homepage/) - Short description, under 250 characters, sentence case. ([Demo](http://url.to/demo), [Source Code](http://url.of/source/code), [Clients](https://url.to/list/of/related/clients-or-apps)) `License` `Language`
  • Additions that depend on proprietary services outside the user's control must be marked .
    - [Name](http://homepage/) `⚠` - Short description, under 250 characters, sentence case. ([Demo](http://url.to/demo), [Source Code](http://url.of/source/code), [Clients](https://url.to/list/of/related/clients-or-apps)) `License` `Language`
  • Additions that are not Free software must be added to non-free.md and have the license set to ⊘ Proprietary
  • Additions are inserted preserving alphabetical order.
  • Additions are not already listed at any of awesome-sysadmin, staticgen.com, staticsitegenerators.net, dbdb.io.
  • The Language tag is the main server-side requirement for the software. Don't include frameworks or specific dialects.
  • Any license you add is in our list of licenses.
  • You have searched the repository for any relevant issues or PRs, including closed ones.
  • Any category you are creating has the minimum requirement of 3 items.
    If not, your addition may be inserted into Misc/Other.
  • Any software project you are adding to the list is actively maintained.
  • Any software project you are adding has working installation instructions.
  • The pull request title is informative, unlike "Update README.md".
    Suggested titles: "Add aaa to bbb" for adding software aaa to section bbb,
    "Remove aaa from bbb" for removing, "Fix license for aaa", etc.

Adding Pairdrop
Copy link
Member

@nodiscc nodiscc left a comment

Choose a reason for hiding this comment

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

Thanks for clearly listing the differences with the original software. That's a lot of nice-looking changes. Have you tried submitting your patches to the original SnapDrop project? Or is there something wrong with the original project/development stopped?

@kgleeson
Copy link
Contributor Author

It's not my project unfortunately, I'm just adding it as it's in more actively development than Snapdrop

@nodiscc
Copy link
Member

nodiscc commented May 17, 2023

Thanks. Looking at https://github.com/RobinLinus/snapdrop/commits/master it seems that the original project has not seen serious maintenance for some time (+ the number of untriaged/unanswered issues), and now concentrates on asking for funding (nothing wrong with that).

Given that the fork has better maintenance and more features, I will edit this PR to remove the original SnapDrop.

@nodiscc nodiscc merged commit 8aff628 into awesome-selfhosted:master May 25, 2023
@kgleeson kgleeson deleted the patch-1 branch May 25, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants