Skip to content

Enable seamless, if crude, file attachment capability - #1544

Open
sgj3 wants to merge 3 commits into
Bitmessage:v0.6from
sgj3:attachments
Open

sgj3 wants to merge 3 commits into
Bitmessage:v0.6from
sgj3:attachments

Conversation

@sgj3

@sgj3 sgj3 commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

See PR #1540.

@sgj3

sgj3 commented Oct 21, 2019

Copy link
Copy Markdown
Contributor Author

This is just a placeholder. I will look into https://bitmessage.org/wiki/Extended_encoding and the src/messagetypes and improve my branch accordingly. For what it's worth, the attachment functionality in this PR already works, even if it is crude and limited.

@g1itch

g1itch commented Oct 22, 2019

Copy link
Copy Markdown
Collaborator

This is just a placeholder. I will look into https://bitmessage.org/wiki/Extended_encoding and the src/messagetypes and improve my branch accordingly. For what it's worth, the attachment functionality in this PR already works, even if it is crude and limited.

Notice the new ability to create draft PR.

No matter which attachment method you use, you should probably check the file size, because of bitmessage message size limit: https://bitmessage.org/wiki/Protocol_specification#object

@dimyme

dimyme commented Nov 14, 2019

Copy link
Copy Markdown
Contributor

attaching files did work here. good job!

@dimyme

dimyme commented Nov 20, 2019 •

Copy link
Copy Markdown
Contributor

However, this is the chance to win the filepicker war.

on many Linuces, you get dozens of filepickers which drive yo nuts. On KDE there is one central filepicker which is ignored by all the loudmouths out there. However, a patch for pyBM was already posted. KDE users will hate non-compliant filepickers such as the Gnome3 one. ideally we should have the patch ready for cherrypicking. seems, in Qt5 , that QtWidgets.QFileDialog.getOpenFileName() automatically honours the KDE filepicker via the "portal..." env setting. time to quit Qt4 !

@dimyme

dimyme commented Nov 29, 2019

Copy link
Copy Markdown
Contributor

using github web site, you can quickly merge this PR into your own fork onto a cherry-branch or sth

works like a charm ! CHERRY 1

@sgj3
sgj3 force-pushed the attachments branch 2 times, most recently from ba16827 to dc2806d Compare February 14, 2020 03:58
sgj3 added 3 commits March 10, 2020 02:41
…presents a data blob, launch a "Save File" dialog and write the file directly, rather than opening the link in an external browser.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants