Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: add cli steps for org sync#15673

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
EdwardAngert merged 9 commits intomainfrom15431-docs-org-sync
Dec 10, 2024
Merged

Conversation

EdwardAngert
Copy link
Contributor

@EdwardAngertEdwardAngert commentedNov 27, 2024
edited
Loading

@EdwardAngertEdwardAngert added the docsArea: coder.com/docs labelNov 27, 2024
@EdwardAngertEdwardAngert self-assigned thisNov 27, 2024
@EdwardAngertEdwardAngert marked this pull request as ready for reviewNovember 27, 2024 18:22
@bpmct
Copy link
Member

@Emyrk - could you review this?

Emyrk reacted with thumbs up emoji

@bpmctbpmct removed their request for reviewDecember 2, 2024 13:45
Copy link
Member

@EmyrkEmyrk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Content is good, just need to change section headings, and add a newDashboard section, making it the default (first).

| mapping | Mapping takes a claim from the IdP, and associates it with 1 or more organizations by UUID. </br> No validation is done, so you can put UUID's of orgs that do not exist (a noop). The UI picker will allow selecting orgs from a drop down, and convert it to a UUID for you. |
| organization_assign_default | This setting exists for maintaining backwards compatibility with single org deployments, either through their upgrade, or in perpetuity. </br> If this is set to 'true', all users will always be assigned to the default organization regardless of the mappings and their IdP claims. |

### Dashboard
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Suggested change
###Dashboard
###Enviroment Config

This readsdashboard, but these docs are configuring via env vars. We should change this title, and have some warning likeNOT RECOMMENDED andwill be overwritten by database settings.

We do need aDashboard section, and that section should be the default.@jaaydenh just recently finished the UI.

Copy link
ContributorAuthor

@EdwardAngertEdwardAngertDec 10, 2024
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@Emyrk I merged in the Dashboard PR, so these steps are now updated - that includes removing the env vars steps

I'll switch their positions so that Dashboard is the first tab

Emyrk reacted with thumbs up emoji
Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

done

2024-12-10_10-48-29

Emyrk reacted with thumbs up emoji
EdwardAngertand others added2 commitsDecember 10, 2024 10:41
[preview](https://coder.com/docs/@15503-ui-org-sync/admin/users/idp-sync#organization-sync-premium)---------Co-authored-by: EdwardAngert <2408959-EdwardAngert@users.noreply.gitlab.com>Co-authored-by: EdwardAngert <17991901+EdwardAngert@users.noreply.github.com>
@EdwardAngertEdwardAngert merged commit5e71992 intomainDec 10, 2024
25 of 26 checks passed
@EdwardAngertEdwardAngert deleted the 15431-docs-org-sync branchDecember 10, 2024 16:09
aslilac pushed a commit that referenced this pull requestDec 11, 2024
[preview](https://coder.com/docs/@15431-docs-org-sync/admin/users/idp-sync#organization-sync-premium)---------Co-authored-by: EdwardAngert <2408959-EdwardAngert@users.noreply.gitlab.com>Co-authored-by: EdwardAngert <17991901+EdwardAngert@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@EmyrkEmyrkEmyrk approved these changes

@jaaydenhjaaydenhAwaiting requested review from jaaydenh

Assignees

@EdwardAngertEdwardAngert

Labels
docsArea: coder.com/docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@EdwardAngert@bpmct@Emyrk

[8]ページ先頭

©2009-2025 Movatter.jp