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

Ability to display custom notification title on Android #437

Conversation

m-ochyra
Copy link
Contributor

@m-ochyra m-ochyra commented Apr 1, 2021

I needed to display title of currently downloaded e-book in notification instead of displaying its file name (which is hash) or URL. This PR adds ability to set notificationTitle param to FlutterDownloader.enqueue().
Zrzut ekranu 2021-04-1 o 11 13 30

@m-ochyra m-ochyra changed the title Ability to display alternative file name in notification on Android Ability to display custom notification title on Android Apr 1, 2021
@m-ochyra
Copy link
Contributor Author

I added also fix for crash on older Android versions:
java.lang.NoSuchMethodError: java.net.HttpURLConnection.getContentLengthLong

@IgeeTheron
Copy link

When will this branch be merged in?

m-ochyra and others added 2 commits June 2, 2021 07:03
# Conflicts:
#	android/src/main/java/vn/hunghd/flutterdownloader/DownloadWorker.java
@knmazur
Copy link

knmazur commented Aug 24, 2021

Is any chance to merge in the nearest time? I also need this feature.

@bartekpacia
Copy link
Collaborator

@m-ochyra Hi, would you be interested in solving the merge conflicts so that we can merge this?

m-ochyra added 3 commits July 19, 2022 08:44
# Conflicts:
#	README.md
#	android/gradle/wrapper/gradle-wrapper.properties
#	android/src/main/java/vn/hunghd/flutterdownloader/DownloadTask.java
#	android/src/main/java/vn/hunghd/flutterdownloader/DownloadWorker.java
#	android/src/main/java/vn/hunghd/flutterdownloader/FlutterDownloaderPlugin.java
#	android/src/main/java/vn/hunghd/flutterdownloader/TaskDao.java
#	android/src/main/java/vn/hunghd/flutterdownloader/TaskDbHelper.java
#	android/src/main/res/xml/provider_paths.xml
#	lib/src/downloader.dart
# Conflicts:
#	android/.idea/compiler.xml
#	android/.idea/gradle.xml
#	android/.idea/misc.xml
@m-ochyra
Copy link
Contributor Author

@m-ochyra Hi, would you be interested in solving the merge conflicts so that we can merge this?

Sure. Done :)

@bartekpacia
Copy link
Collaborator

@m-ochyra Thanks! 😎

bartekpacia added a commit that referenced this pull request Aug 18, 2022
…" (#707)

* Revert "Ability to display custom notification title on Android (#437)"

This reverts commit 5e80a37.

* set version to 1.8.3
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.

4 participants