Align with other similar properties and use "uids" naming, since we're at it
have it manipulated as a string list since that's what it really is.
Details
Details
- Reviewers
dfaure bensi - Commits
- R4:d78532fb1ecd: Adjust the context list property
Diff Detail
Diff Detail
- Repository
- R4 Zanshin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
The code changes are all about using a different name for the property and a different type (QStringList instead of QString), right? The property already existed.
The commit log is very confusing because it doesn't say that, it makes it sound like this was not handled before, and the commit now handles this by adding a new property.
Comment Actions
Right, that's because in fact I authored this before your own patch. :-)
It was stuck in my own queue because I wanted to validate it with the following commit which I struggled with. I'll change the description.
Comment Actions
Did the arc amend locally to reflect the description change, that should address your comment.