Skip to content
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

Update teamID #9300

Closed
wants to merge 1 commit into from
Closed

Update teamID #9300

wants to merge 1 commit into from

Conversation

luacmartins
Copy link
Contributor

Details

Updates the teamID, since Desktop notarization is failing with the error:

Error: HTTP status code: 403. Invalid or inaccessible developer team ID for the provided Apple ID. Ensure the Team ID is correct and that you are a member of that team.
  failedTask=build stackTrace=Error: Failed to notarize via notarytool

From the developer account, it seems that teamID is 368M544MTT

Screen Shot 2022-06-02 at 3 55 40 PM

Fixed Issues

$ #9277

Tests

  1. Merge this PR
  2. Verify the staging build is notarized correctly

@luacmartins luacmartins self-assigned this Jun 2, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2022

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@luacmartins luacmartins marked this pull request as ready for review June 2, 2022 21:56
@luacmartins luacmartins requested a review from a team as a code owner June 2, 2022 21:56
@melvin-bot melvin-bot bot requested review from amyevans and removed request for a team June 2, 2022 21:57
@luacmartins luacmartins requested review from a team and removed request for AndrewGable, roryabraham and amyevans June 2, 2022 21:57
@melvin-bot melvin-bot bot removed the request for review from a team June 2, 2022 21:57
@AndrewGable
Copy link
Contributor

I think this is a dupe of #9299, right?

@melvin-bot melvin-bot bot requested a review from amyevans June 2, 2022 21:57
@luacmartins
Copy link
Contributor Author

Ah lol seems like @AndrewGable beat me to it #9299. Closing.

@luacmartins luacmartins closed this Jun 2, 2022
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.

2 participants