R2 Maintenance Release #544
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Removes default arg values in `<InfoField>` - Adds args to InfoField story
…claimant experience (#469) - Do not display 500 for missing Application Insights environment variable - Do not throw errors if logger is not working
Bumps [tmpl](https://github.com/daaku/nodejs-tmpl) from 1.0.4 to 1.0.5. - [Release notes](https://github.com/daaku/nodejs-tmpl/releases) - [Commits](https://github.com/daaku/nodejs-tmpl/commits/v1.0.5) --- updated-dependencies: - dependency-name: tmpl dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [prismjs](https://github.com/PrismJS/prism) from 1.24.1 to 1.25.0. - [Release notes](https://github.com/PrismJS/prism/releases) - [Changelog](https://github.com/PrismJS/prism/blob/master/CHANGELOG.md) - [Commits](PrismJS/prism@v1.24.1...v1.25.0) --- updated-dependencies: - dependency-name: prismjs dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [nth-check](https://github.com/fb55/nth-check) from 2.0.0 to 2.0.1. - [Release notes](https://github.com/fb55/nth-check/releases) - [Commits](fb55/nth-check@v2.0.0...v2.0.1) --- updated-dependencies: - dependency-name: nth-check dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [semver-regex](https://github.com/sindresorhus/semver-regex) from 3.1.2 to 3.1.3. - [Release notes](https://github.com/sindresorhus/semver-regex/releases) - [Commits](https://github.com/sindresorhus/semver-regex/commits) --- updated-dependencies: - dependency-name: semver-regex dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Global Header Icons properly highlight on hover Bugfix - Global Header link post context root change wontfix: our spacing doesn't quite match due to our centering. Okay'd by design. hides all of this on main!
Co-authored-by: Rocket <[email protected]>
…'/claimstatus' unless overridden.
…pino child logger (#523) - Assigns a UUID to each request - Creates a [pino child logger](https://github.com/pinojs/pino/blob/master/docs/child-loggers.md) for each request to include the UUID in all logger calls - Adds an [`AsyncLocalStorage`](https://nodejs.org/api/async_context.html#class-asynclocalstorage) shared utility - Adds additional unit tests for logging calls - If pino is undefined, still pass the logging message through to `console.log()`
- Updates ansi-regex from 5.0.0 to 5.0.1
- Updates chromatic from 5.7.1 to 6.0.5
#525) - Decouples the static asset path from whether the app is running in an Azure environment vs a non-Azure environment - Introduces a new optional environment variable `ASSET_PREFIX` that allows us to more easily control the asset prefix (i.e. Azure environments that don't have Front Door url rewriting can still work) - By default, the static asset prefix is `/claimstatus` and can be overriden to be something else (typically `""`)
rocketnova
commented
Nov 8, 2021
Comment on lines
+11
to
+12
| "uio-desktop-home-url": "https://uio.edd.ca.gov/UIO/Pages/ExternalUser/ClaimantAccountManagement/UIOnlineHome.aspx", | ||
| "uio-mobile-home-url": "https://uiom.edd.ca.gov/UIOM/Pages/Public/ExternalUser/UIOMobileLandingPage.aspx", |
There was a problem hiding this comment.
Note: I noticed these links differ from each other (desktop vs mobile) so I went back to look through my emails. I can confirm that UIO confirmed via email on Sept 29 and Sept 30 (thread subject "RE: Update CST Test environment for global header demo") that this is the correct behavior. We implemented it as part of #505
lomky
reviewed
Nov 9, 2021
| let globalHeader: JSX.Element | ||
|
|
||
| /* eslint-disable no-constant-condition */ | ||
| if (true) { |
There was a problem hiding this comment.
Just confirming that we are releasing the code for Global Header, but it is hardocded inactive here.
lomky
approved these changes
Nov 9, 2021
There was a problem hiding this comment.
LGTM! 
Tue Nov 09 16:05:51 [production] ~/repos/ui-claim-tracker $ git pull
Already up to date.
Tue Nov 09 16:06:37 [production] ~/repos/ui-claim-tracker $ git co main && git pull
Switched to branch 'main'
Your branch is up to date with 'origin/main'.
Already up to date.
Tue Nov 09 16:06:42 [main] ~/repos/ui-claim-tracker $ git log --first-parent --oneline production..main
29a66b5 (HEAD -> main, origin/main) Update application insights to 2.1.8 (#537)
1a14c3f Add a comment to avoid throwing errors from within logger.log (#536)
48390ea Add an environment variable for more flexibility for static asset path (#525)
8e9b3c6 Update to latest chromatic (#533)
17c5766 Update ansi-regex (#535)
f803ae6 Log a UUID for all traces in a single request/response cycle using a pino child logger (#523)
97c30f3 Adds an error catch for null responses
62bfe42 Fine tunes & then hides the Global Header on main
f871c44 Run CI on future and PRs against future & production
a40fbf1 Flexible context root
c89a69c Creates the Global Header
d79ff60 Bump semver-regex from 3.1.2 to 3.1.3 (#499)
6e5dfae Bump nth-check from 2.0.0 to 2.0.1 (#498)
5183e49 Bump prismjs from 1.24.1 to 1.25.0 (#497)
273f813 Bump tmpl from 1.0.4 to 1.0.5 (#496)
3bd7615 Security update: set-value (#495)
f48d2cf Adds storybooks for Maintenance Component & Page under Maintenance
d30ff5e Update dependencies (#488)
2be865a Missing Application Insights environment variable should not disrupt claimant experience (#469)
5ea5fe3 Update modheader instructions in readme
0f32698 Remove default argument values in InfoField (#463)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release
mainintoproductionChangelog
View the changelog of 21 PRs in this release using: