Skip to content

feat: put deployments into the user's default project#86

Merged
colinnielsen merged 1 commit intomainfrom
colinnielsen/mp-434-prompt-the-user-to-associate-a-deployment-to-their-account
Sep 26, 2023
Merged

feat: put deployments into the user's default project#86
colinnielsen merged 1 commit intomainfrom
colinnielsen/mp-434-prompt-the-user-to-associate-a-deployment-to-their-account

Conversation

@colinnielsen
Copy link
Copy Markdown
Contributor

Background

Checklist

  • Documentation updated
  • Tested code changes

@linear
Copy link
Copy Markdown

linear bot commented Sep 26, 2023

MP-434 Pass auth between CLI -> preview-service requests

Metro import, preview, and deploy should send authorization headers to the backend with their deployment request, it should add it to their default project in their user org.

  • [CLI] Add authorization headers to POST requests to /deploy if the ID token exists
    • Add to $ metro import
    • Add to $ metro deploy
    • Add to $ metro preview
  • [PRV:SRV] load the user based on authorization headers
  • [PRV:SRV] add a project?: stringflag to the DeployRequestParams
  • [PRV:SRV] validate the project exists if specified and add it, otherwise add it to the default project

@its-everdred
Copy link
Copy Markdown
Contributor

Just have 2 conflicts

@colinnielsen colinnielsen force-pushed the colinnielsen/mp-434-prompt-the-user-to-associate-a-deployment-to-their-account branch from bd6e05d to ed35386 Compare September 26, 2023 22:37
@colinnielsen colinnielsen changed the title Colinnielsen/mp-434-prompt-the-user-to-associate-a-deployment-to-their-account feat: put deployments into the user's default project Sep 26, 2023
@colinnielsen colinnielsen merged commit eac2393 into main Sep 26, 2023
@colinnielsen colinnielsen deleted the colinnielsen/mp-434-prompt-the-user-to-associate-a-deployment-to-their-account branch September 26, 2023 22:48
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