Skip to content

Conversation

@jablan
Copy link
Collaborator

@jablan jablan commented Oct 28, 2025

@hahmed-dev
Copy link
Contributor

hahmed-dev commented Oct 28, 2025

I guess we can wait on this PR? Until we have a final decision on how to proceed about this problem? Maybe we decide to add a new attribute?

@jablan
Copy link
Collaborator Author

jablan commented Nov 12, 2025

@hahmed-dev I think it's not very realistic to add another field any time soon. I'd suggest to make this field behave consistently.

@hahmed-dev
Copy link
Contributor

Yes @jablan I understand, it will be a bigger refactoring, but given the fact that right now this field is not behaving consistently, should we wait with this PR to avoid any further confusion? Or maybe make it explicitly from where user_id actually changes (not from uploads) but thats not a good way... So maybe we can wait for this to be merged until we have clarity on the behaviour of the field?

title: translation_details
properties:
user:
description: The user who last updated the translation content
Copy link
Member

Choose a reason for hiding this comment

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

I believe we can only phrase it like that if this would be true for all cases, which is currently not the case. So we need to wait with this change.

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