Skip to content

Conversation

epipav
Copy link
Collaborator

@epipav epipav commented Sep 8, 2022

Changes proposed ✍️

  • Github worker now checks if the repo is available with given token first and adds the repo to the endpoints array if repo is available.

Checklist ✅

  • Label appropriately with type:feature 🚀, type:enhancement ✨, type:bug 🐞, or type:documentation 📜.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated
    • Front-end: frontend/.env.dist
    • Backend: backend/.env.dist, backend/.env.dist.staging, backend/.env.dist.staging.
    • Configuration docs have been updated.
    • Team members only: update environment variables in Password manager and update the team
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@netlify
Copy link

netlify bot commented Sep 8, 2022

Deploy Preview for open-devfounders canceled.

Name Link
🔨 Latest commit 22814c8
🔍 Latest deploy log https://app.netlify.com/sites/open-devfounders/deploys/632031fdf7c43a000973fe72

@netlify
Copy link

netlify bot commented Sep 8, 2022

Deploy Preview for open-crowd-prod canceled.

Name Link
🔨 Latest commit 22814c8
🔍 Latest deploy log https://app.netlify.com/sites/open-crowd-prod/deploys/632031fd339baf0009ba0138

@joanreyero joanreyero changed the title Bugfix/GitHub check repo available GitHub integration: check repo can be queried before starting parsing Sep 8, 2022
@epipav
Copy link
Collaborator Author

epipav commented Sep 13, 2022

baseIterator changes are already reviewed by @joanreyero Rest of the functionality (that checks if gh repo is available or not) is unit tested and tested out in staging as well.

@epipav epipav merged commit 4b580a5 into main Sep 13, 2022
@epipav epipav deleted the bugfix/github-check-repo-available branch September 13, 2022 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant