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

feat(sentry): add SENTRY_PROJECT & SENTRY_URL #363

Merged
merged 2 commits into from Aug 15, 2023

Conversation

JoeKarow
Copy link
Contributor

@JoeKarow JoeKarow commented Aug 2, 2023

Overview

Added configuration for SENTRY_PROJECT & SENTRY_URL env vars. [Docs]

Type of change

  • Created a new plugin
  • Improved an existing plugin
  • Fixed a bug in an existing plugin
  • Improved contributor utilities or experience

Related Issue(s)

  • Resolves: #
  • Relates: #

How To Test

Run sentry-cli info. The configured values for SENTRY_PROJECT & SENTRY_URL should show:

Sentry Server: https://sentry.io
Default Organization: acme
Default Project: example

Authentication Info:
  Method: Auth Token
  User: user@example.com
  Scopes:
    - event:admin
    - event:read
    - member:read
    - org:read
    - project:read
    - project:releases
    - team:read

Changelog

Added support for SENTRY_PROJECT & SENTRY_URL.

@arunsathiya arunsathiya self-requested a review August 2, 2023 14:55
@arunsathiya arunsathiya added the waiting-on-reviewer signals that a certain PR is waiting for a review from a 1Password team member label Aug 9, 2023
Copy link
Contributor

@arunsathiya arunsathiya left a comment

Choose a reason for hiding this comment

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

These changes look good to me, and functional test successful. Thanks for the PR, @JoeKarow!

Copy link
Contributor

@AndyTitu AndyTitu left a comment

Choose a reason for hiding this comment

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

lgtm

@arunsathiya arunsathiya added waiting-on-sec-review and removed waiting-on-reviewer signals that a certain PR is waiting for a review from a 1Password team member labels Aug 11, 2023
@jpcoenen
Copy link
Member

Thanks for your contribution 🙌 Looks good!

@arunsathiya arunsathiya merged commit c352bda into 1Password:main Aug 15, 2023
3 checks passed
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.

None yet

4 participants