Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STCOR-910 provide IfAnyPermission, stripes.ifAnyPerm #1560

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Conversation

zburke
Copy link
Member

@zburke zburke commented Nov 14, 2024

<IfPermission> and stripes.ifPerm() return true when the user has ALL the given permissions. <IfAnyPermission> and stripes.ifAnyPerm() return true when the user has ANY of the given permissions.

Refs STCOR-910

`<IfPermission>` and `stripes.ifPerm()` return `true` when the user has
ALL the given permissions. `<IfAnyPermission>` and `stripes.ifAnyPerm()`
return `true` when the user has ANY of the given permission.

Refs STCOR-910
Copy link

Bigtest Unit Test Results

192 tests  ±0   187 ✅ ±0   6s ⏱️ ±0s
  1 suites ±0     5 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit aed7bbc. ± Comparison against base commit 0d65a69.

This pull request removes 5 and adds 3 tests. Note that renamed tests count towards both.
      equal to check email label in english translation
      equal to check email precautions label in english translation
      equal to sent email precautions label in english translation
Chrome_130_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the header with an appropriate text content
Chrome_130_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
Chrome_130_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the header with an appropriate text content
      equal to check email label in english translation
Chrome_130_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
      equal to check email precautions label in english translation
Chrome_130_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
      equal to sent email precautions label in english translation

Copy link

Jest Unit Test Results

  1 files  ±0   57 suites  +2   1m 35s ⏱️ -1s
351 tests +8  351 ✅ +8  0 💤 ±0  0 ❌ ±0 
354 runs  +8  354 ✅ +8  0 💤 ±0  0 ❌ ±0 

Results for commit aed7bbc. ± Comparison against base commit 0d65a69.

Copy link
Contributor

@BogdanDenis BogdanDenis left a comment

Choose a reason for hiding this comment

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

Thank you @zburke!

@zburke zburke merged commit 921f0ab into master Nov 14, 2024
16 checks passed
@zburke zburke deleted the STCOR-910 branch November 14, 2024 14:13
zburke added a commit that referenced this pull request Nov 15, 2024
`<IfPermission>` and `stripes.ifPerm()` return `true` when the user has
ALL the given permissions. `<IfAnyPermission>` and `stripes.ifAnyPerm()`
return `true` when the user has ANY of the given permissions.

Refs STCOR-910

(cherry picked from commit 921f0ab)
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