| SBisson | |
| Sep 27 2022, 1:29 PM |
| F35627738: image.png | |
| Oct 25 2022, 10:46 PM |
| F35627733: image.png | |
| Oct 25 2022, 10:46 PM |
Steps to replicate the issue (include links if applicable):
What happens?:
story_already_exists is always false
What should have happened instead?:
story_already_exists should be false for a new story and true for an existing story
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Instrumentation: log story_already_exist correctly | mediawiki/extensions/Wikistories | master | +23 -18 |
Change 835631 had a related patch set uploaded (by Sbisson; author: Sbisson):
[mediawiki/extensions/Wikistories@master] Instrumentation: log story_already_exist correctly
Change 835631 merged by jenkins-bot:
[mediawiki/extensions/Wikistories@master] Instrumentation: log story_already_exist correctly
Hi @nshahquinn-wmf , would you like to have a look at this to confirm the changes to the instrumentation?
An existing story has the story_already_exists as true and a new one has the flag as false.
 |  |
Closing this now.
Sorry for the delay! I've just checked the data we've received for this and there are a number of events where story_already_exists. I checked a random sample and the logged values were all correct. We can safely close this.