Add selectRepos mutation

hyperlog-core / hyperlog-backend

Backend for hyperlog

0 stars 0 forks source link

Add selectRepos mutation #82

Closed nikochiko closed 4 years ago

nikochiko commented 4 years ago

Changes:

Add selectRepos mutation which sets the given list as the selectedRepos attribute on DynamoDB profile analysis table
Remove check for max turns from analyse Mutation

nikochiko commented 4 years ago

Do take a look at the condition expressions part. I've made it so that the selectedRepos cannot be overwritten, I tried to add a condition so that all repos in the given list should be already present in the repos Map using the Contains expression but I'm not sure if it would work for Map type. Lmk if there are any corrections there

BrainBuzzer commented 4 years ago

What kind of mutation should I give? I've tried this one, but it doesn't work.

mutation{
  selectRepos(repoNames:["hyperlog/cli", "hyperlog/hyperlog-frontend"]){
    success
    errors
  }
}

nikochiko commented 4 years ago

I just noticed one typo with that, so now the argument should be repos: [...] instead of repoNames, other than that it's supposed to work. I just noticed another error with boto3 while trying it out. I'll fix it and get back