Skip to content

refactor: refactoring framework - #382

Draft
medcl wants to merge 141 commits into
mainfrom
console_framework
Draft

refactor: refactoring framework #382
medcl wants to merge 141 commits into
mainfrom
console_framework

Conversation

@medcl

@medcl medcl commented Jun 11, 2026

Copy link
Copy Markdown
Member

UserSessionInfo.Permissions was removed but modules/security/account/refresh.go still referenced it, causing compilation failures.

Changes

  • Removed direct Permissions field usage in refreshSessionUser and cloneSessionUser
  • Replaced with NewUserAssignedPermission(GetAllPermissionsForUser(sessionUser), nil) — the same pattern used in access_token/authentication.go and core/security/cache.go
sessionUser.UserAssignedPermission = security.NewUserAssignedPermission(
    security.GetAllPermissionsForUser(sessionUser), nil,
)

hardy-dev-infinilabs and others added 30 commits May 6, 2026 17:18
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…lias

GetPipelineTasksResponse should be an alias of GetPipelinesResponse, not the other way around. The primary type GetPipelinesResponse is the concrete map type.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI changed the title Console framework fix: remove references to deleted Permissions field in UserSessionInfo Jun 11, 2026
Copilot finished work on behalf of medcl June 11, 2026 09:29
@medcl medcl changed the title fix: remove references to deleted Permissions field in UserSessionInfo refactor: refactoring framework Jun 11, 2026
@medcl
medcl marked this pull request as draft June 16, 2026 10:27
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.

3 participants