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

fix: Added envLoggingUrl in webpack #294

Merged
merged 3 commits into from
Apr 18, 2024
Merged

Conversation

vsrivatsa-edinburgh
Copy link
Member

@vsrivatsa-edinburgh vsrivatsa-edinburgh commented Apr 16, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Added envLoggingUrl to consume logging url from env in webpack if available

Motivation and Context

How did you test it?

set a custom logging url
export envLoggingUrl=https://example.com
observe that the logs are pushed to the custom endpoint

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

@prafulkoppalkar prafulkoppalkar merged commit 88587ac into main Apr 18, 2024
3 checks passed
@github-actions github-actions bot added the Closed Label will be automatically added when the PR will get merged to main label Apr 18, 2024
akash-c-k pushed a commit that referenced this pull request Apr 18, 2024
## [0.43.2](v0.43.1...v0.43.2) (2024-04-18)

### Bug Fixes

* Added envLoggingUrl in webpack ([#294](#294)) ([88587ac](88587ac))
rising-dragon401 added a commit to rising-dragon401/hyperswitch-web that referenced this pull request Jul 2, 2024
## [0.43.2](juspay/hyperswitch-web@v0.43.1...v0.43.2) (2024-04-18)

### Bug Fixes

* Added envLoggingUrl in webpack ([#294](juspay/hyperswitch-web#294)) ([88587ac](juspay/hyperswitch-web@88587ac))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants