Skip to content
This repository has been archived by the owner on Nov 19, 2021. It is now read-only.

Metadata: Update not triggering with modifying comment bodies #1382

Open
GuiltyDolphin opened this issue Jun 1, 2016 · 6 comments
Open

Metadata: Update not triggering with modifying comment bodies #1382

GuiltyDolphin opened this issue Jun 1, 2016 · 6 comments
Labels

Comments

@GuiltyDolphin
Copy link
Member

GuiltyDolphin commented Jun 1, 2016

It looks like modifying the body of issues/comments isn't triggering the metadata to update (it is supported).

For example, duckduckgo/zeroclickinfo-goodies#2105 previously did not link to the Calculator's IA page, did not show up on the IA page, and only appeared in the 'other' issues on the Contributor Profile (it did not appear under the IA-specific issues).

After updating the issue to reference the IA page, waiting, and doing hard-refereshes, the status has not updated on the IA page or profile. This is also the case with many other IAs.

/cc @alohaas @jdorweiler

@alohaas
Copy link
Contributor

alohaas commented Jun 1, 2016

@MariagraziaAlastra @jagtalon do either of you have input on this issue?
Let us know if you need more description or screen shots.

@alohaas alohaas added the Bug label Jun 1, 2016
@MariagraziaAlastra
Copy link
Member

@GuiltyDolphin that's because we look for issues and PRs opened/closed/updated (where updated means new comments) within the last hour, and just changing the description would still make the issue fall off the radar.
I closed it and reopened it, this should be enough to update the IA Page whenever ghIssues.pl will run next time.
And since that's how we get issues linked to IAs on the profiles as well, it should reflect there too when the data gets generated again by the cronjob 😄

@GuiltyDolphin
Copy link
Member Author

@MariagraziaAlastra Check out https://duckduckhack.com/u/guiltydolphin/ (go to the issues/PRs section, at the bottom); there's a bunch of PRs that need updating (I've added the links).

Perhaps it might be worth running something over old issues that were created before we started adding the IA page as given (after updating them ofc)?

@MariagraziaAlastra
Copy link
Member

Right, there isn't a specific period of time we can tackle, or not limited enough that we can hope not to bust our GH API limit, that's the problem.

@GuiltyDolphin
Copy link
Member Author

GuiltyDolphin commented Jun 1, 2016

@MariagraziaAlastra Does changing labels update it? If so I can just add/remove labels when the page has been updated (alternatively opening/closing might be fine - hope no one gets antsy!)

@MariagraziaAlastra
Copy link
Member

Nope unfortunately

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants