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

Rename the column with a torrent's creation date from "Updated" to "Created" to avoid confusion #7340

Merged

Conversation

kozlovsky
Copy link
Contributor

@kozlovsky kozlovsky commented Mar 29, 2023

This PR renames the column with the torrent's creation date from "Updated" to "Created" to avoid user confusion in the UI and adds the corresponding translation to all languages.

The corresponding data item field is still named "updated" in the application code. It is not changed to avoid accidental bugs in application logic and can be renamed later.

@kozlovsky kozlovsky changed the title Fix/UI rename updated to created Rename the column with a torrent's creation date from "Updated" to "Created" to avoid confusion Mar 29, 2023
@kozlovsky kozlovsky marked this pull request as ready for review March 29, 2023 08:48
@kozlovsky kozlovsky requested review from a team, drew2a and xoriole and removed request for a team March 29, 2023 08:48
@drew2a
Copy link
Contributor

drew2a commented Mar 29, 2023

The corresponding data item field is still named "updated" in the application code. It is not changed to avoid accidental bugs in application logic and can be renamed later.

Could you please create a corresponding issue? Overwise there is a risk to forget.

@kozlovsky
Copy link
Contributor Author

I was able to rename it from "updated" to "created" in all necessary places, it turned out to be easier than I expected :)

@kozlovsky kozlovsky force-pushed the fix/ui_rename_updated_to_created branch from 22b2a58 to 3cc5800 Compare March 29, 2023 12:01
@kozlovsky kozlovsky requested a review from drew2a March 29, 2023 12:44
@kozlovsky kozlovsky merged commit 037d839 into Tribler:release/7.13 Mar 29, 2023
@kozlovsky kozlovsky deleted the fix/ui_rename_updated_to_created branch March 29, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants