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: #557 Configure CLIENT API Token For DEMO #558

Merged

Conversation

ianliuwk1019
Copy link
Collaborator

@ianliuwk1019 ianliuwk1019 commented Jan 10, 2024

Configuring "DEMO" deployment to use CLIENT "PROD" for batch data refreshing.


Thanks for the PR!

Any successful deployments (not always required) will be available below.

Once merged, code will be promoted and handed off to following workflow run.

@ianliuwk1019 ianliuwk1019 marked this pull request as ready for review January 10, 2024 17:50
@ianliuwk1019 ianliuwk1019 merged commit 3c1bc1e into main Jan 10, 2024
17 checks passed
@ianliuwk1019 ianliuwk1019 deleted the fix/557-config-demo-forest-client-prod-data-refresh branch January 10, 2024 22:08
Copy link
Collaborator

@basilv basilv left a comment

Choose a reason for hiding this comment

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

looks fine.

@basilv
Copy link
Collaborator

basilv commented Jan 11, 2024

Weird, I'm trying to approve the PR but can't.

@basilv
Copy link
Collaborator

basilv commented Jan 11, 2024

(I don't have the option in GitHub to approve the PR.)

@ianliuwk1019
Copy link
Collaborator Author

ianliuwk1019 commented Jan 12, 2024

(I don't have the option in GitHub to approve the PR.)

Oh, sorry @basilv I just saw this. The PR is merged and closed yesterday. Thanks for looking at this, I thought you were very busy so Catherine approved it and I merged it and closed it.

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.

3 participants