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-898] Allow RTR configuration to be runtime-configurable #1544

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

ncovercash
Copy link
Member

Split from STCOR-888 (#1535), this allows RTR components to react to changes in the global Stripes configuration.

@ncovercash ncovercash requested a review from zburke October 10, 2024 21:14
Copy link

github-actions bot commented Oct 10, 2024

Jest Unit Test Results

  1 files  ±0   56 suites  ±0   1m 1s ⏱️ +2s
339 tests ±0  339 ✅ ±0  0 💤 ±0  0 ❌ ±0 
345 runs  +2  345 ✅ +2  0 💤 ±0  0 ❌ ±0 

Results for commit 50c1291. ± Comparison against base commit 9f28052.

This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
configureRtr leaves existing settings in place ‑ configureRtr leaves existing settings in place
configureRtr sets idleSessionTTL and idleModalTTL ‑ configureRtr sets idleSessionTTL and idleModalTTL
FFetch class force refresh event Invokes a refresh on RTR_FORCE_REFRESH_EVENT... ‑ FFetch class force refresh event Invokes a refresh on RTR_FORCE_REFRESH_EVENT...
configureRtr sets default values as applicable ‑ configureRtr sets default values as applicable

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Oct 10, 2024

Bigtest Unit Test Results

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

Results for commit 50c1291. ± Comparison against base commit 9f28052.

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_129_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_129_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_129_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_129_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_129_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

♻️ This comment has been updated with latest results.

Copy link

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.

1 participant