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

Change default download location #1983

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Change default download location #1983

merged 1 commit into from
Jun 20, 2024

Conversation

jubalh
Copy link
Member

@jubalh jubalh commented Jun 20, 2024

/url save $someurl will now download to
~/.local/share/profanity/downloads/from_jid/date/filename instead of ~/.local/share/profanity/downloads.

Like this the downloaded files should be better ordered.

This will only happen for MUC and 1:1 chat windows. Private windows might have only a nick or jid. Lets not distinguish between those two ways. And since the nick option is unreliable to be the same person lets just put them in the general downloads folder. As is done with downloads from all other windows as well.

I also had the idea to make this configurable but this suits my needs and time limits right now.

`/url save $someurl` will now download to
`~/.local/share/profanity/downloads/from_jid/date/filename` instead of
`~/.local/share/profanity/downloads`.

Like this the downloaded files should be better ordered.

This will only happen for MUC and 1:1 chat windows.
Private windows might have only a nick or jid. Lets not distinguish
between those two ways. And since the nick option is unreliable to be
the same person lets just put them in the general downloads folder.
As is done with downloads from all other windows as well.

I also had the idea to make this configurable but this suits my needs
and time limits right now.
@jubalh jubalh added the behaviour-change changes to current behaviour label Jun 20, 2024
@jubalh jubalh added this to the next milestone Jun 20, 2024
@jubalh jubalh self-assigned this Jun 20, 2024
@jubalh jubalh merged commit 4d9852a into master Jun 20, 2024
5 checks passed
@jubalh jubalh deleted the feat/dl-location branch June 20, 2024 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
behaviour-change changes to current behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant