feat: add organizations field to support synchronization of multi-organization users #3005
Conversation
|
@tangyang9464 @imp2002 please review |
Bsheepcoder
changed the title
| @@ -30,7 +30,7 @@ require ( | |||
| github.com/go-telegram-bot-api/telegram-bot-api v4.6.4+incompatible | |||
| github.com/go-webauthn/webauthn v0.6.0 | |||
| github.com/golang-jwt/jwt/v4 v4.5.0 | |||
| github.com/google/uuid v1.4.0 | |||
| github.com/google/uuid v1.6.0 | |||
There was a problem hiding this comment.
imho, it is unnecessary to change go.mod , it will be better to open a new pr to upgrade packages
There was a problem hiding this comment.
ok, i get it,thanks
There was a problem hiding this comment.
@Bsheepcoder not next time. It needs to be fixed "this time", or this PR will not pass
| @@ -23,6 +23,7 @@ | |||
| "antd": "5.2.3", | |||
| "antd-token-previewer": "^1.1.0-22", | |||
| "buffer": "^6.0.3", | |||
| "caniuse-lite": "^1.0.30001621", | |||
There was a problem hiding this comment.
Maybe not. I seem to have added it when executing'go mod tidy 'or other commands. I'll pay attention next time.
There was a problem hiding this comment.
@Bsheepcoder not next time. It needs to be fixed "this time", or this PR will not pass
| errCh := make(chan error, len(owners)) | ||
|
|
||
| for _, owner := range owners { | ||
| owner := owner // 创建局部变量，避免闭包中的问题 |
| @@ -30,7 +30,7 @@ require ( | |||
| github.com/go-telegram-bot-api/telegram-bot-api v4.6.4+incompatible | |||
| github.com/go-webauthn/webauthn v0.6.0 | |||
| github.com/golang-jwt/jwt/v4 v4.5.0 | |||
| github.com/google/uuid v1.4.0 | |||
| github.com/google/uuid v1.6.0 | |||
There was a problem hiding this comment.
@Bsheepcoder not next time. It needs to be fixed "this time", or this PR will not pass
| @@ -100,6 +106,9 @@ class SyncerEditPage extends React.Component { | |||
|
|
|||
| const syncer = this.state.syncer; | |||
| syncer[key] = value; | |||
| if (key === "organization") { | |||
| syncer.organizations = []; // 清空 organizations 字段 | |||
| @@ -5623,6 +5623,11 @@ caniuse-lite@^1.0.0, caniuse-lite@^1.0.30001464, caniuse-lite@^1.0.30001503: | |||
| resolved "https://registry.yarnpkg.com/caniuse-lite/-/caniuse-lite-1.0.30001507.tgz#fae53f6286e7564783eadea9b447819410a59534" | |||
| integrity sha512-SFpUDoSLCaE5XYL2jfqe9ova/pbQHEmbheDf5r4diNwbAgR3qxM9NQtfsiSscjqoya5K7kFcHPUQ+VsUkIJR4A== | |||
|
|
|||
| caniuse-lite@^1.0.30001621: | |||
There was a problem hiding this comment.
not next time. It needs to be fixed "this time", or this PR will not pass
| @@ -297,6 +297,7 @@ | |||
| "Organization": "Organization", | |||
| "Organization - Tooltip": "Similar to concepts such as tenants or user pools, each user and application belongs to an organization", | |||
| "Organizations": "Organizations", | |||
| "Organizations - Tooltip": "Provides multi-organization synchronization capabilities", | |||
|
Closed for now |
Fix : #2891