Page MenuHomePhabricator

[Story] Set the image Property id in configurations
Closed, ResolvedPublic

Description

The image property is needed in the EntityRenderer and should be set via the settings.

Event Timeline

Lucie claimed this task.
Lucie raised the priority of this task from to Medium.
Lucie updated the task description. (Show Details)
Lucie added subscribers: hoo, gerritbot, Lucie.
Lydia_Pintscher added a subscriber: Lydia_Pintscher.

This should be configured on-wiki, not in the server config of the wiki.

This should be configured on-wiki, not in the server config of the wiki.

I disagree… it can still be overwritten in the Wiki, but we should be able to set a default for all Wikis in the config.

We provide the default config just like we provide the default Lua modules. This shouldn't be different.

We provide the default config just like we provide the default Lua modules. This shouldn't be different.

Sure, but that shouldn't be hardcoded in the extension as the extension is supposed to be Wikidata-agnostic (it only knows about Wikibase).

We will have a number of property lists that will depend on Wikidata. The other ones are for example the ranking of properties and the blacklist of properties. We need a sane and easily editable (by the editors) way to let them change and maintain it.

I don't think making this completely independent of Wikidata is necessary. Easy editability by Wikipedia editors is more important.

Change 256439 had a related patch set uploaded (by Lucie Kaffee):
Set the image Property id in configurations

https://gerrit.wikimedia.org/r/256439

Change 256439 merged by jenkins-bot:
Set the image Property id in configurations

https://gerrit.wikimedia.org/r/256439

hoo moved this task from Doing to Done on the ArticlePlaceholder board.
hoo removed a project: Patch-For-Review.