Skip to content

fix 503 error from opencollective #4214

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Nov 28, 2020

Conversation

chenxsan
Copy link
Member

closes #4211

@vercel
Copy link

vercel bot commented Nov 27, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/webpack-docs/webpack-js-org/noz4x80yh
✅ Preview: https://webpack-js-org-git-bugfix-fix-opencollective-503.webpack-docs.now.sh

@chenxsan chenxsan changed the title use api key for opencollective fix 503 error from opencollective Nov 27, 2020
transactions(limit: $limit, offset: $offset, includeIncognitoTransactions: false) {
nodes {
// only query transactions in last year
const transactionsGraphqlQuery = `query transactions($dateFrom: ISODateTime, $limit: Int, $offset: Int) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now we query only data in last year.

@chenxsan chenxsan marked this pull request as ready for review November 27, 2020 09:14
@chenxsan chenxsan requested a review from sokra November 27, 2020 09:15
@chenxsan chenxsan merged commit 4de31ad into webpack:master Nov 28, 2020
@chenxsan chenxsan deleted the bugfix/fix-opencollective-503 branch November 28, 2020 04:22
@@ -42,6 +42,7 @@ jobs:
uses: JamesIves/[email protected]
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
OPENCOLLECTIVE_API_KEY: ${{ secrets.OPENCOLLECTIVE_API_KEY }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which account holds this key?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer used, could be removed. PR here #4390

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

Successfully merging this pull request may close these issues.

503 error when running fetch:supporters on CI
3 participants