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

update to new service instance #4825

Merged
merged 1 commit into from
Oct 3, 2024
Merged

update to new service instance #4825

merged 1 commit into from
Oct 3, 2024

Conversation

jgw96
Copy link
Contributor

@jgw96 jgw96 commented Oct 3, 2024

fixes #4823

PR Type

Bugfix

Describe the current behavior?

Service was broken, unfortunately I must be light on details here.

Describe the new behavior?

Pointing to our new instance of the main test service.

PR Checklist

  • [x ] Test: run npm run test and ensure that all tests pass
  • [ x] Target main branch (or an appropriate release branch if appropriate for a bug fix)
  • [ x] Ensure that your contribution follows standard accessibility guidelines. Use tools like https://webhint.io/ to validate your changes.

Additional Information

Copy link
Contributor

Thanks jgw96 for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

@@ -17,7 +17,7 @@ export const env = {
//@ts-ignore
if (import.meta.env.PROD) {
env.isProduction = true;
env.api = 'https://pwabuilder-apiv2-node.azurewebsites.net/api';
env.api = 'https://pwabuilder-apiv2-container.calmflower-2e2ebb94.eastus.azurecontainerapps.io/api';
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

@jgw96 jgw96 enabled auto-merge October 3, 2024 17:45
@jgw96 jgw96 disabled auto-merge October 3, 2024 17:45
@jgw96 jgw96 merged commit f9d285c into main Oct 3, 2024
6 of 7 checks passed
@jgw96 jgw96 deleted the update-to-new-service branch October 3, 2024 17:45
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.

PWABuilder is currently down
2 participants