Skip to content

Conversation

@Mr-Leshiy
Copy link
Contributor

Description

Applying the latest rbac-registration crate version, with the updated interface and moved all stateful validation logic to the library.

Related Issue(s)

Part of #3507

Description of Changes

Provide a clear and concise description of what the pull request changes.

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@Mr-Leshiy Mr-Leshiy requested a review from apskhem November 18, 2025 09:41
@Mr-Leshiy Mr-Leshiy added squad: gatekeepers Catalyst App Backend, System Development & Integration Team review me PR is ready for review labels Nov 18, 2025
@Mr-Leshiy Mr-Leshiy moved this from New to 👀 In review in Catalyst Nov 20, 2025
@github-actions
Copy link
Contributor

📚 Docs Preview

The docs for this PR can be previewed at the following URL:

https://docs.dev.projectcatalyst.io/voices/feat/latest-rbac-crate

Comment on lines +186 to +190
let stake_addresses: HashSet<_> = new_chain
.stake_addresses()
.difference(&previous_addresses)
.cloned()
.collect();
Copy link
Member

Choose a reason for hiding this comment

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

Note: this is one of the places that we need to update (in the future) to handle removed stake addresses.

@Mr-Leshiy Mr-Leshiy merged commit 49bf0a6 into feature/new-rbac Nov 21, 2025
46 checks passed
@Mr-Leshiy Mr-Leshiy deleted the feat/latest-rbac-crate branch November 21, 2025 05:28
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Catalyst Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review me PR is ready for review squad: gatekeepers Catalyst App Backend, System Development & Integration Team

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants