Haven't got a digest in a while #65

steveoh · 2018-10-02T18:02:18Z

22 days. Did something change? Do I need to modify my config or something? repo is https://github.com/agrc/gis.utah.gov

brad-decker · 2018-10-06T21:37:33Z

Confirmed it stopped working for us too.

abhijeetps · 2018-10-08T14:34:45Z

Hi @steveoh , @brad-decker ,
Thanks for opening this issue.
Please have a look at this: https://github.com/search?o=desc&q=Weekly+Digest&s=created&type=Issues

You might be seeing that Weekly Digest is working fine for these repositories.

Is it possible to share the repository where you have installed the application so that I can have a look at it?

Try reinstalling app in your repository once. Also, the configuration is optional. Weekly Digest will work fine without configuring .github/weekly-digest.yml.

If the problem persists, please let me know so that I can trace the bug behind it. Currently, I cannot retrace the bug.

Please suggest some more features that you would like to see in the weekly digest app. 😄

steveoh · 2018-10-08T16:35:24Z

I left the repo link in my comment. I'll try the uninstall reinstall option

steveoh · 2018-10-10T01:43:44Z

I think my issue is that you changed publish day from an integer to a day abbreviation at some point or that is the only difference i can see right now.

abhijeetps · 2018-10-10T07:57:53Z

@steveoh , Yeah, I saw that you made some changes in that repository: agrc/gis.utah.gov@b389164

And looks like it's working fine now: agrc/gis.utah.gov#962 🎉

I made the app to understand both numeric and alphabetical dates and work properly, but I am not sure what went wrong. I will look again into it. 👀

Sorry for the inconvenience caused. 🙏

abhijeetps · 2018-10-10T07:58:56Z

@brad-decker , maybe you might be facing the same problem, can you once check it out and update if the problem persists.
Thanks. 😄

brad-decker · 2018-10-15T15:55:13Z

So i deleted the install, then reinstalled it. Same issue. I got the initial weekly digest but then sunday came and went without an update. is a config file now required? @abhijeetps

steveoh · 2018-10-15T16:03:52Z

Mine is set to publish monday so if nothing shows up tonight or tomorrow morning, then there is an issue up stream.

abhijeetps · 2018-10-15T16:07:53Z

@brad-decker , Adding weekly-digest.yml for config is an optional feature. It works by default if the configuration is not defined.

@steveoh , I am watching your repository closely, the app should publish weekly digest, if it won't, I will ask the my mentor about the issue.

abhijeetps · 2018-10-15T16:13:03Z

Here's the recent Weekly Digest, being published by app on many repositories (on Sunday), for example: https://github.com/search?o=desc&q=Weekly+Digest&s=created&type=Issues

maestromac · 2018-10-15T17:48:25Z

I'm also not getting the weekly update. I receive the first digest and that was it. This is me fiddling around with the setting trying to evoke it. Any tips?

maestromac · 2018-10-29T15:56:43Z

re-adding solved it. This bug might have something to do with updating the .yml file.

steveoh · 2018-10-29T16:16:28Z

I haven't got a digest in a while. What was your solution?

maestromac · 2018-10-29T16:20:45Z

What I did was.

Remove .github/weekly-digest.yml
Remove the Weekly Digest integration
Re-add Weekly Digest integration
Re-add .github/weekly-digest.yml

steveoh · 2018-10-29T20:12:07Z

Ok I just tried your steps. We'll see how my mileage varies.

steveoh · 2018-11-07T18:15:30Z

Still no digest. I don't know how to make this work. What is special about my repo?

maestromac · 2018-11-07T18:31:11Z

Mine stop working too ☹️

abhijeetps · 2018-11-07T18:45:15Z

I have opened a discussion here on Slack: https://probot-talk.slack.com/archives/C6S4FC8AF/p1541616078059300
I too can't figure out the reason behind the app not working fine on few repositories. I hope someone from Probot will help me with this.
I will let you know if I receive any updates.

Slack

abhijeetps · 2018-11-19T07:33:38Z

Hey, just a minor update on my progress of trying to figure out why the app is not working on some repositories.
I was checking out logs and found the following error:

ERROR event: {"message":"API rate limit exceeded for installation ID XXXXXX.","documentation_url":"https://developer.github.com/v3/#rate-limiting"}
  HttpError: {"message":"API rate limit exceeded for installation ID XXXXXX.","documentation_url":"https://developer.github.com/v3/#rate-limiting"}
      at response.text.then.message (/home/nowuser/src/node_modules/@octokit/rest/lib/request/request.js:72:19)
      at process._tickCallback (internal/process/next_tick.js:68:7)
  
  event: {
    "event": "schedule.repository",
    "installation": XXXXXX,
    "repository": "owner/repository"
  }

I will try to figure out with Probot and GitHub team of how I can work with this problem.
As the popularity of this app is increasing, this app might need more requests in future, and needs a strong solution for the same.

I will keep you updated about this more.
Thank you for using Weekly Digest app. 😄

steveoh · 2019-01-09T15:57:31Z

Any love on this lately?

yangwenmai · 2019-01-18T02:24:23Z

talkgo/night#123
After that, I modified the default configuration, but it didn't trigger the build, and then tried to reinstall, but it's still not generated. Because the default value is Sunday, so I need to wait until Sunday to trigger?

yangwenmai · 2019-01-20T12:22:39Z

@abhijeetps Please help me when you're not busy.

yangwenmai · 2019-01-27T04:32:15Z

@abhijeetps Please help me when you're not busy.

Now, It's work for my repos

abhijeetps · 2019-01-28T04:50:15Z

@yangwenmai , it's really good to see that it works on your repository.
Though, probably you might still face some problem with it as Sunday is a pretty busy day for Weekly Digest app.
I recommend you to add .github/weekly-digest.yml file and configure your app to publish weekly digest on some other day.

abhijeetps · 2019-01-28T04:55:09Z

@steveoh , I know you were among the first users of my Weekly Digest application, and I feel really bad that it's not working on your repo. :(
I have seen that you have even tried changing days but still, it's not working out.
But, one thing that I am pretty sure of is that it not a problem in your end, there's some bug in the app only which is holding it from publishing weekly digests for your repo.
I am trying to find out a bug for the same, but still, I have no clue, what the bug is. I think I will have to try debug the code and spend a day at the logs to check the problem behind it. 😄

abhijeetps · 2019-01-29T13:24:47Z

@steveoh , looks like Weekly Digest app published a Weekly Digest on Monday as you have configured the app to be. Thus, IMO, to sum it up, I think it's probably because it's touching rate limits before and it's unable to publish digests.
agrc/gis.utah.gov#1014
Let's see what happens on next Monday.

abhijeetps · 2019-01-29T13:27:12Z

@maestromac and @brad-decker , is it possible to share your repo name where you have installed Weekly Digest app so that I can have a close look on the problem.
And probably, it looks like it published a digest on @steveoh repository, might have worked on your as well.

yangwenmai · 2019-01-29T13:57:00Z

@yangwenmai , it's really good to see that it works on your repository.
Though, probably you might still face some problem with it as Sunday is a pretty busy day for Weekly Digest app.
I recommend you to add .github/weekly-digest.yml file and configure your app to publish weekly digest on some other day.

I set thurday, and i will take a look in that day.

abhijeetps · 2019-02-04T12:46:09Z

Hey @steveoh , sorry for the ping again.
But it looks like the Weekly Digest app has started working fine on your repository again for the second time from my last message.
See: agrc/gis.utah.gov#1020

Since it has started working again normally without any changes done on my end, I think I can say that it was some problem with GitHub APIs.

I will close this issue now.

Also, I have a piece of good news that I would like to share. Week;y Digest now has over 5000 Weekly Digests published by the bot! 🎉

Thank you all for your love and support for my app! ❤️
Also, please feel free to reopen a new issue, if a new bug arrives.

abhijeetps added the bug Something isn't working label Oct 8, 2018

steveoh closed this as completed Oct 10, 2018

steveoh reopened this Oct 29, 2018

abhijeetps closed this as completed Feb 4, 2019

abhijeetps mentioned this issue May 9, 2019

Why not to run Weekly_Digest in last week #74

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Haven't got a digest in a while #65

Haven't got a digest in a while #65

steveoh commented Oct 2, 2018

brad-decker commented Oct 6, 2018

abhijeetps commented Oct 8, 2018

steveoh commented Oct 8, 2018

steveoh commented Oct 10, 2018 •

edited

Loading

abhijeetps commented Oct 10, 2018

abhijeetps commented Oct 10, 2018

brad-decker commented Oct 15, 2018

steveoh commented Oct 15, 2018

abhijeetps commented Oct 15, 2018

abhijeetps commented Oct 15, 2018

maestromac commented Oct 15, 2018

maestromac commented Oct 29, 2018

steveoh commented Oct 29, 2018

maestromac commented Oct 29, 2018

steveoh commented Oct 29, 2018

steveoh commented Nov 7, 2018

maestromac commented Nov 7, 2018

abhijeetps commented Nov 7, 2018 •

edited by unfurl-links bot

Loading

abhijeetps commented Nov 19, 2018

steveoh commented Jan 9, 2019

yangwenmai commented Jan 18, 2019

yangwenmai commented Jan 20, 2019

yangwenmai commented Jan 27, 2019

abhijeetps commented Jan 28, 2019 •

edited

Loading

abhijeetps commented Jan 28, 2019

abhijeetps commented Jan 29, 2019

abhijeetps commented Jan 29, 2019

yangwenmai commented Jan 29, 2019

abhijeetps commented Feb 4, 2019

Haven't got a digest in a while #65

Haven't got a digest in a while #65

Comments

steveoh commented Oct 2, 2018

brad-decker commented Oct 6, 2018

abhijeetps commented Oct 8, 2018

steveoh commented Oct 8, 2018

steveoh commented Oct 10, 2018 • edited Loading

abhijeetps commented Oct 10, 2018

abhijeetps commented Oct 10, 2018

brad-decker commented Oct 15, 2018

steveoh commented Oct 15, 2018

abhijeetps commented Oct 15, 2018

abhijeetps commented Oct 15, 2018

maestromac commented Oct 15, 2018

maestromac commented Oct 29, 2018

steveoh commented Oct 29, 2018

maestromac commented Oct 29, 2018

steveoh commented Oct 29, 2018

steveoh commented Nov 7, 2018

maestromac commented Nov 7, 2018

abhijeetps commented Nov 7, 2018 • edited by unfurl-links bot Loading

abhijeetps commented Nov 19, 2018

steveoh commented Jan 9, 2019

yangwenmai commented Jan 18, 2019

yangwenmai commented Jan 20, 2019

yangwenmai commented Jan 27, 2019

abhijeetps commented Jan 28, 2019 • edited Loading

abhijeetps commented Jan 28, 2019

abhijeetps commented Jan 29, 2019

abhijeetps commented Jan 29, 2019

yangwenmai commented Jan 29, 2019

abhijeetps commented Feb 4, 2019

steveoh commented Oct 10, 2018 •

edited

Loading

abhijeetps commented Nov 7, 2018 •

edited by unfurl-links bot

Loading

abhijeetps commented Jan 28, 2019 •

edited

Loading