news-catalyst / next-tinynewsdemo

Frontend for Tiny News Collective organizations
https://next-tinynewsdemo.vercel.app
1 stars 0 forks source link

Testing one more org with a different GitHub env API call #1269

Closed chelsea-tnc closed 2 years ago

chelsea-tnc commented 2 years ago

I want to test whether the API call we currently use is causing issues with the GA API key. So I'm gonna swap it out for a different API call that does the same thing.

In bootstrap.js, I locally changed lines 212-221 from:

const secretResult = await octokit.rest.actions.createOrUpdateEnvironmentSecret(
        {
          repository_id: repoID,
          environment_name: environmentName,
          secret_name: secretName,
          encrypted_value: encryptedValue,
          key_id: pubKeyID,
        }

to:

const secretResult = await octokit.request(
        'PUT /repositories/{repository_id}/environments/{environment_name}/secrets/{secret_name}',
        {
          repository_id: repoID,
          environment_name: environmentName,
          secret_name: secretName,
          encrypted_value: encryptedValue,
          key_id: pubKeyID,
        }
      );

Then I ran the boostrap script locally. The script went through as expected. Now, I'm uploading this new org's yml file to see if the workflow will run. If it does, that means the API we were previously using was not correctly saving the API key.

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
staging-tiny-news-sites ✅ Ready (Inspect) Visit Preview Jul 14, 2022 at 6:15PM (UTC)
9 Ignored Deployments | Name | Status | Preview | Updated | | :--- | :----- | :------ | :------ | | **ang-diaryo** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/ang-diaryo/AuQL9auVQKifdvmTk4R92cEz6kBR)) | | Jul 14, 2022 at 6:15PM (UTC) | | **austin-vida** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/austin-vida/2yZs795GNjEtjP23g7Wj8khdsw7i)) | | Jul 14, 2022 at 6:15PM (UTC) | | **black-by-god** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/black-by-god/BGjBaZKvMG3GMfSJsiWjN31p8gh1)) | | Jul 14, 2022 at 6:15PM (UTC) | | **five-wards-media** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/five-wards-media/5ZGS31DddTq1B7VqxDNM8SZqYVLE)) | | Jul 14, 2022 at 6:15PM (UTC) | | **harvey-world-herald** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/harvey-world-herald/55QKbUZecKpsxzkVZs5YHwXbFnAi)) | | Jul 14, 2022 at 6:15PM (UTC) | | **next-tinynewsdemo** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/next-tinynewsdemo/28oZX3j748hfrm8Po7Qm9kQNPfc3)) | | Jul 14, 2022 at 6:15PM (UTC) | | **spotlight-schools** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/spotlight-schools/Hgj7QC1vDji6nvY7u9aTstLmBiBS)) | | Jul 14, 2022 at 6:15PM (UTC) | | **tiny-news-curriculum** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/tiny-news-curriculum/26XWfGsZ6SgfTuXq7GM3aSJkvtQy)) | | Jul 14, 2022 at 6:15PM (UTC) | | **tiny-news-sites** | ⬜️ Ignored ([Inspect](https://vercel.com/news-catalyst/tiny-news-sites/CYjGMyoZcfpyKpjVqJZ4jUsTqjno)) | | Jul 14, 2022 at 6:15PM (UTC) |
cypress[bot] commented 2 years ago



Test summary

28 0 0 0


Run details

Project TNC Front-End
Status Passed
Commit 5231c061cd ℹ️
Started Jul 14, 2022 6:17 PM
Ended Jul 14, 2022 6:21 PM
Duration 04:05 💡
OS Linux Ubuntu - 20.04
Browser Firefox 102

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard