Skip to content

Feat/verify trees admin - #1234

Open
Mohmn wants to merge 3 commits into
Greenstand:keycloakfrom
Mohmn:feat/verify-trees-admin
Open

Feat/verify trees admin#1234
Mohmn wants to merge 3 commits into
Greenstand:keycloakfrom
Mohmn:feat/verify-trees-admin

Conversation

@Mohmn

@Mohmn Mohmn commented Jul 30, 2026

Copy link
Copy Markdown
Member

Description

[Add a description of the changes]

Issue(s) addressed

  • Resolves #issue_number

What kind of change(s) does this PR introduce?

  • Enhancement
  • Bug fix
  • Refactor

Please check if the PR fulfils these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Issue

What is the current behavior?

What is the new behavior?

Breaking change

Does this PR introduce a breaking change?

Other useful information

@Mohmn
Mohmn requested a review from a team as a code owner July 30, 2026 18:41
@Mohmn

Mohmn commented Jul 30, 2026

Copy link
Copy Markdown
Member Author

1 similar comment
@Mohmn

Mohmn commented Jul 30, 2026

Copy link
Copy Markdown
Member Author

@dadiorchen

Copy link
Copy Markdown
Collaborator

@Mohmn
For the admin feature, I can see there is logic to check the capture in list:

Then('I should be able to verify the first tree', async () => {
  await VerifyPage.verifyFirstCapture();
});

But for the organization, I can not find any code to check: there is no tree/capture in the verify page? I suppose we need to do that so we can say: the list is filtered correctly by the role/permission of the user.

@Mohmn

Mohmn commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

@Mohmn For the admin feature, I can see there is logic to check the capture in list:

Then('I should be able to verify the first tree', async () => {
  await VerifyPage.verifyFirstCapture();
});

But for the organization, I can not find any code to check: there is no tree/capture in the verify page? I suppose we need to do that so we can say: the list is filtered correctly by the role/permission of the user.

updated the bdd

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.

2 participants