Skip to content

Conversation

@mcdurdin
Copy link
Member

Relates-to: keymanapp/keyman.com#558
Relates-to: keymanapp/shared-sites#55
Relates-to: keymanapp/shared-sites#56

TODO:

  • After shared-sites PR56 merges, update BOOTSTRAP_VERSION

@mcdurdin mcdurdin added this to the B18S3 milestone Mar 13, 2025
@mcdurdin mcdurdin added the chore label Mar 13, 2025
@mcdurdin mcdurdin requested a review from darcywong00 March 13, 2025 10:23
Co-authored-by: Darcy Wong <darcy_wong@sil.org>
Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin mcdurdin merged commit 6496a74 into master Mar 14, 2025
2 checks passed
@mcdurdin mcdurdin deleted the chore/use-simpler-bootstrap-and-new-sentry branch March 14, 2025 07:05
@github-project-automation github-project-automation bot moved this to Done in Keyman Mar 14, 2025
@mcdurdin
Copy link
Member Author

Unfortunately, tier seems to have been lost on the deployed version. Need to figure out why:

Sentry initialization complete: tier=undefined

@mcdurdin
Copy link
Member Author

Unfortunately, tier seems to have been lost on the deployed version.

This appears to just be the log message that is wrong. Will still fix, but less critical:

  console.log(`Sentry initialization complete: tier=${sentryEnvironment.tier}`);

Everywhere else correctly uses sentryEnvironment.environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants