Skip to content

feat: Show Api V2 title if available#600

Merged
sirambd merged 5 commits intoupload-v2from
show-v2-title
Mar 6, 2026
Merged

feat: Show Api V2 title if available#600
sirambd merged 5 commits intoupload-v2from
show-v2-title

Conversation

@sirambd
Copy link
Member

@sirambd sirambd commented Mar 6, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for displaying (and entering) a transfer title when available (notably for API v2 users), while keeping the created-date fallback for older transfers.

Changes:

  • Add a new localized string for the transfer title field label.
  • Expose an isApiV2Flow and show a transfer title input only for API v2 users; make sender email read-only for API v2.
  • Display transfer.title (fallback to created date) in transfer list items and the transfer details top app bar.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
app/src/main/res/values/strings.xml Adds default string for the transfer title field label.
app/src/main/res/values-sv/strings.xml Adds Swedish translation for the transfer title field label.
app/src/main/res/values-pt/strings.xml Adds Portuguese translation for the transfer title field label.
app/src/main/res/values-pl/strings.xml Adds Polish translation for the transfer title field label.
app/src/main/res/values-nl/strings.xml Adds Dutch translation for the transfer title field label.
app/src/main/res/values-nb/strings.xml Adds Norwegian Bokmål translation for the transfer title field label.
app/src/main/res/values-it/strings.xml Adds Italian translation for the transfer title field label.
app/src/main/res/values-fr/strings.xml Adds French translation for the transfer title field label.
app/src/main/res/values-fi/strings.xml Adds Finnish translation for the transfer title field label.
app/src/main/res/values-es/strings.xml Adds Spanish translation for the transfer title field label.
app/src/main/res/values-el/strings.xml Adds Greek translation for the transfer title field label.
app/src/main/res/values-de/strings.xml Adds German translation for the transfer title field label.
app/src/main/res/values-da/strings.xml Adds Danish translation for the transfer title field label.
app/src/main/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/pickfiles/PickFilesViewModel.kt Injects AccountUtils, derives isApiV2Flow, and updates author email from the current user.
app/src/main/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/pickfiles/PickFilesScreen.kt Adds UI for title input (API v2 only) and makes sender email read-only for API v2.
app/src/main/java/com/infomaniak/swisstransfer/ui/screen/main/transferdetails/TransferDetailsScreen.kt Uses transfer title (fallback to created date) in the top app bar.
app/src/main/java/com/infomaniak/swisstransfer/ui/previewparameter/GroupedTransfersPreviewParameterProvider.kt Updates preview data to include a title field.
app/src/main/java/com/infomaniak/swisstransfer/ui/components/transfer/TransferItem.kt Shows transfer title (fallback to created date) in list items.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Base automatically changed from download-with-password to upload-v2 March 6, 2026 13:50
@sirambd sirambd requested review from tevincent and removed request for LunarX March 6, 2026 13:56
@Infomaniak Infomaniak deleted a comment from Copilot AI Mar 6, 2026
…ewparameter/GroupedTransfersPreviewParameterProvider.kt

Co-authored-by: Gibran Chevalley <32095402+LunarX@users.noreply.github.com>
@sirambd sirambd requested review from LunarX and tevincent March 6, 2026 14:56
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2026

@sirambd sirambd removed the request for review from tevincent March 6, 2026 16:02
@sirambd sirambd merged commit 8d1e3c7 into upload-v2 Mar 6, 2026
6 of 7 checks passed
@sirambd sirambd deleted the show-v2-title branch March 6, 2026 16:07
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