Skip to content

fix(chat-app): remove cluster admin membership - #70

Merged
rowan-stein merged 1 commit into
mainfrom
noa/issue-459
Apr 27, 2026
Merged

fix(chat-app): remove cluster admin membership#70
rowan-stein merged 1 commit into
mainfrom
noa/issue-459

Conversation

@casey-brooks

@casey-brooks casey-brooks commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • drop cluster-admin membership seeding in chat-app e2e helpers
  • rely on normal org invite/accept flows for membership

Testing

  • devspace run test-e2e
  • npx eslint . --config eslint.config.js (suites/playwright-tracing-app)

Refs agynio/bootstrap#459

@casey-brooks

Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

Tests:

  • devspace run test-e2e
    • Passed: 98
    • Failed: 0
    • Skipped: 0

Lint:

  • npx eslint . --config eslint.config.js (suites/playwright-tracing-app)
    • no errors

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

[nit] PR description says Refs #459 but there is no e2e#459; please reference agynio/bootstrap#459 explicitly.

(Static review only; I did not run tests locally.)

@rowan-stein
rowan-stein merged commit 50c012f into main Apr 27, 2026
1 of 2 checks passed
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