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

(need help) Cover image feature #6278

Closed
wants to merge 24 commits into from
Closed

(need help) Cover image feature #6278

wants to merge 24 commits into from

Conversation

tellmewhy12
Copy link

@tellmewhy12 tellmewhy12 commented May 13, 2021

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • add cover image to downloaded files
  • maybe add title and artist metadata later
  • does not yet have any actual implementation, only the preferences exist (see below!)

Fixes the following issue(s)

Relies on the following changes

  • Change in the Download Preferences
  • Change in the Download Dialogue
  • Change in the Download Manager <--- HERE i need help, where should the code get implemented? how can i use the video / audio file that gets downloaded in the code? can i make changes to the giga download manager?

APK testing

On the website the APK can be found by going to the "Checks" tab below the title and then on "artifacts" on the right.

Due diligence

@triallax
Copy link
Contributor

Please don't change the README; it's unrelated to your changes.

@triallax triallax added downloader Issue is related to the downloader feature request Issue is related to a feature in the app labels May 13, 2021
@@ -715,4 +715,8 @@
<string name="night_theme_summary">Select your favorite night theme — %s</string>
<string name="select_night_theme_toast">You can select your favorite night theme below</string>
<string name="download_has_started">Download has started</string>
<string name="cover_image">Cover Image</string>
<string name="download_cover_preference_title">Initially add cover image</string>
<string name="download_cover_preference">cover_pref_key</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

This particular line should go in settings_keys.xml, and you should set translatable="false" on it.

@Stypox
Copy link
Member

Stypox commented Jun 7, 2021

@mangoepic this can't be done without adding ffmpeg, see the discussion in #5445

@Stypox Stypox closed this Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
downloader Issue is related to the downloader feature request Issue is related to a feature in the app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add metadata while downloading files mp3/m4a cover pic feature request
4 participants