Skip to content

Commit

Permalink
Merge pull request #1546 from ardriveapp/PE-5356-integrate-sentry-for…
Browse files Browse the repository at this point in the history
…-error-logging-in-production-builds

PE-5356: feat(logging)
  • Loading branch information
thiagocarvalhodev authored Jan 23, 2024
2 parents 9eaa0da + 5910385 commit eec8c05
Show file tree
Hide file tree
Showing 12 changed files with 262 additions and 365 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ env:
PR_NUMBER: ${{ github.event.pull_request.number }}
PR_TITLE: ${{ github.event.pull_request.title }}
RELEASE_NOTES: ${{ github.event.pull_request.title }} - ${{ github.sha }}

jobs:
pre-build:
uses: ./.github/workflows/test.yml
Expand Down Expand Up @@ -42,7 +42,7 @@ jobs:
run: |
scr setup
flutter config --enable-web
flutter build web --dart-define=environment=development --release --pwa-strategy=none --no-web-resources-cdn
flutter build web --dart-define=environment=development --release --pwa-strategy=none --no-web-resources-cdn --source-maps
# JS files cache invalidation
- name: main.dart.js and service worker cache invalidation
Expand Down
8 changes: 7 additions & 1 deletion .github/workflows/production.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,11 @@ on:
env:
BUILD_NUMBER: ${{ github.run_number }}
RELEASE_NOTES: 'Production'
SENTRY_DSN: ${{secrets.SENTRY_DSN}}
SENTRY_ORG: ${{secrets.SENTRY_ORG}}
SENTRY_AUTH_TOKEN: ${{secrets.SENTRY_AUTH_TOKEN}}
SENTRY_PROJECT: ${{secrets.SENTRY_PROJECT}}


jobs:
pre-build:
Expand Down Expand Up @@ -44,7 +49,8 @@ jobs:
run: |
scr setup
flutter config --enable-web
flutter build web --release --dart-define=environment=production --pwa-strategy=none --no-web-resources-cdn
flutter build web --release --dart-define=environment=production --dart-define=SENTRY_DSN=${SENTRY_DSN} --dart-define=SENTRY_PROJECT=${SENTRY_PROJECT} --dart-define=SENTRY_AUTH_TOKEN=${SENTRY_AUTH_TOKEN} --dart-define=SENTRY_ORG=${SENTRY_ORG} --pwa-strategy=none --no-web-resources-cdn
flutter packages pub run sentry_dart_plugin
# Disribute to Firebase
- uses: FirebaseExtended/action-hosting-deploy@v0
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/staging.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ on:
env:
BUILD_NUMBER: ${{ github.run_number }}
RELEASE_NOTES: Staging - ${{ github.sha }}
SENTRY_DSN: ${{secrets.SENTRY_DSN}}
SENTRY_ORG: ${{secrets.SENTRY_ORG}}
SENTRY_AUTH_TOKEN: ${{secrets.SENTRY_AUTH_TOKEN}}
SENTRY_PROJECT: ${{secrets.SENTRY_PROJECT}}

jobs:
pre-build:
Expand Down Expand Up @@ -46,7 +50,8 @@ jobs:
run: |
scr setup
flutter config --enable-web
flutter build web --dart-define=environment=staging --release --pwa-strategy=none --no-web-resources-cdn
flutter build web --dart-define=environment=staging --dart-define=SENTRY_DSN=${SENTRY_DSN} --dart-define=SENTRY_PROJECT=${SENTRY_PROJECT} --dart-define=SENTRY_AUTH_TOKEN=${SENTRY_AUTH_TOKEN} --dart-define=SENTRY_ORG=${SENTRY_ORG} --release --pwa-strategy=none --no-web-resources-cdn
flutter packages pub run sentry_dart_plugin
# Deploy to github pages
- uses: JamesIves/[email protected]
Expand Down
Loading

0 comments on commit eec8c05

Please sign in to comment.