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.

Sign up for GitHub

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

Jump to bottom

feat(spa,api): add sentry for observablity #180

Merged
merged 27 commits into from
May 18, 2023
Merged

feat(spa,api): add sentry for observablity #180

merged 27 commits into from
May 18, 2023

Conversation

timonmasberg
Copy link
Member

@timonmasberg timonmasberg commented May 2, 2023 •
edited
Loading

  • introduces Sentry for SPA and API with a separate Observability library for each app that is active for non-CI production environments
  • minor refactoring and fixes in the CI workflows (e.g. now building all project every time)

@timonmasberg timonmasberg self-assigned this May 2, 2023
@timonmasberg timonmasberg reopened this May 2, 2023
@timonmasberg timonmasberg marked this pull request as ready for review May 2, 2023 13:43
@timonmasberg timonmasberg reopened this May 2, 2023
@timonmasberg timonmasberg marked this pull request as draft May 2, 2023 13:44
@timonmasberg timonmasberg force-pushed the chore/telemetry branch 11 times, most recently from 602f8b7 to 2164837 Compare May 3, 2023 07:21
@github-actions github-actions bot deleted a comment from timonmasberg May 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023 •
edited
Loading

🚀 Deployment Preview

SPA: https://jolly-glacier-057c0e703-180.westeurope.2.azurestaticapps.net
API: https://kordis-api-pr180.azurewebsites.net
Commit SHA: de4c6af

@timonmasberg timonmasberg marked this pull request as ready for review May 3, 2023 08:04
@timonmasberg timonmasberg requested a review from JSPRH May 3, 2023 08:04
@timonmasberg timonmasberg force-pushed the chore/telemetry branch 4 times, most recently from 2983bb2 to 88492ea Compare May 6, 2023 15:06
@timonmasberg timonmasberg marked this pull request as draft May 6, 2023 15:16
@sonarcloud
Copy link

sonarcloud bot commented May 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

89.3% 89.3% Coverage
0.0% 0.0% Duplication

@timonmasberg timonmasberg marked this pull request as ready for review May 6, 2023 17:16
@timonmasberg
ci: temporarily bypass prettier checks
d144be7
@timonmasberg
ci: workaround for faster npm ci npm/cli#2011
ea48494
@timonmasberg
ci: list different for prettier checks
b1aec8d
@timonmasberg
ci: change nx format check to prettier check
a415074
@timonmasberg
chore: fix auth service spec
f49cc26
@timonmasberg
chore: fix trace wrapper spec
5512591
@timonmasberg
chore(spa): add sentry key to environment template
2214a70
@timonmasberg
chore(api): remove unused code from resolver trace wrapper test
945fa8a
@timonmasberg
ci(api): add sentry release to build and deploy action
c615e3e
@timonmasberg
ci: fix incorrect OAuth secret name
bdcc7f5
@timonmasberg
ci: fix prettier warning on success
099b88d
@timonmasberg
chore: fix code smells
da090a4
@timonmasberg
chore: commets and explanations
c2d54a5
@timonmasberg
ci: pass Sentry auth token via input to build actions
366c6a8
@timonmasberg
fix: async functions not traced correctly
5c0e4b3
@timonmasberg
fix(spa): sentry module initialized in CI environments
77bce68
@timonmasberg
chore(spa): add noop observability service in dev environments
db7bb34
@timonmasberg
chore: recreate lockfile after rebase
a3a2d74
@timonmasberg
chore: migrate with changes from main
48fc58a
@timonmasberg
chore: rebase
d46ad52
@timonmasberg
chore: code review suggestions
7ab14ee
@timonmasberg
chore: apply migrations
99b7907
@sonarcloud
Copy link

sonarcloud bot commented May 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

90.7% 90.7% Coverage
0.0% 0.0% Duplication

@timonmasberg timonmasberg enabled auto-merge (squash) May 18, 2023 11:01
@timonmasberg timonmasberg merged commit 312183a into main May 18, 2023
9 checks passed
@timonmasberg timonmasberg deleted the chore/telemetry branch May 18, 2023 11:02
@timonmasberg
Copy link
Member Author

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

@JSPRH JSPRH JSPRH left review comments

Assignees

@timonmasberg timonmasberg

Labels
released
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@timonmasberg @JSPRH