Skip to content

Add local release date for movie#800

Open
ArthurTakase wants to merge 1 commit intomichaldrabik:masterfrom
ArthurTakase:feat/localRelease
Open

Add local release date for movie#800
ArthurTakase wants to merge 1 commit intomichaldrabik:masterfrom
ArthurTakase:feat/localRelease

Conversation

@ArthurTakase
Copy link
Contributor

@ArthurTakase ArthurTakase commented Mar 14, 2024

(I'm redoing the PR after cleaning my fork)

Select release date region in app settings (same setting as network region).
Update release date when region change.
Add region flag next to local release date in movie details.

  • If the local date is identical to the original release, both flags are displayed.
  • If the local date is different, both dates are displayed, each with its flag.
  • If no local date exists, this does not change from the current app.

Modification of "findBestImage" to give priority to posters from the user's region (movies and series).
Added a flag to indicate the origin country of shows.

image image

@michaldrabik
Copy link
Owner

Thanks will be looking but I have few other things to finish first.

However - for any translation related issues are changes please use CrowdIn https://crowdin.com/project/showly-android-app :)

Select release date region in app settings.
Add region flag next to local release date in movie details.
If the local date is identical to the original release, both flags are displayed.
If the local date is different, both dates are displayed, each with its flag.
If no local date exists, this does not change the current app.
Update release date when region change.
handle when no date
use TMDB for more accurate dates
poster from selected region

FIX: merge
@ArthurTakase
Copy link
Contributor Author

ArthurTakase commented Jan 2, 2025

Hello @michaldrabik, I hope you're doing well. Are there any changes I can make to this PR to ensure the feature can be made available in the application? Thank you in advance for your feedback!

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.

2 participants