- add store module
- copy user name from window
Description
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Lydia_Pintscher | T224833 first client edit for simple typo | |||
Resolved | Lydia_Pintscher | T226999 Save Wikidata edit when submitting client edit modal | |||
Resolved | • Matthias_Geisler_WMDE | T230335 Create SavingEntityRepository | |||
Declined | • Matthias_Geisler_WMDE | T230331 Add user module incl user name to store |
Event Timeline
Change 533180 had a related patch set uploaded (by Matthias Geisler; owner: Matthias Geisler):
[mediawiki/extensions/Wikibase@master] bridge: Makeing ForeinRepo responsible
Actually we do not need a user store module and for the user name can be part of the save repo until we have a state changing behavior for users. That is the reason why the actual patch does not include a store change at all. I would love some different thoughts on that!
I totally agree with the approach of making the user name a repository constructor argument instead of putting it in the store somewhere.
I’m tempted to say that we should close this task as Declined and reattach the Gerrit change to T230335: Create SavingEntityRepository – after all, it now implements something completely different from what this task describes.
Change 533180 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: Makeing ForeignRepo responsible for holding username