-
Notifications
You must be signed in to change notification settings - Fork 3
Work on improving the create service principal generator #90
Conversation
🦋 Changeset detectedLatest commit: aa5f406 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -101,8 +101,30 @@ pnpm nx g \ | |||
@arkahna/nx-terraform:add-project-environment \ | |||
<projectname> \ | |||
--environment <environmentname> | |||
``` | |||
|
|||
### create-environment-sp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added some doco, it's now doing a fair bit
|
||
## Concepts | ||
|
||
![Concepts](./docs/concepts.drawio.png) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a visual around how it ends up causing the projects to be structured will be handy
'--display-name', | ||
servicePrincipalName, | ||
]) | ||
const appObjectId: string = JSON.parse(stdoutAppList)[0].id |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It now gathers and outputs the AD Object ID as well as the app id and the app Object ID.
Different commands need one of these ids.
It then stores all 3 in the dev markdown.
No description provided.