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 server_config.yaml #22

Merged
merged 2 commits into from
May 17, 2024
Merged

Update server_config.yaml #22

merged 2 commits into from
May 17, 2024

Conversation

anshulg954
Copy link
Collaborator

@anshulg954 anshulg954 commented May 17, 2024

Change Description

Brief (a few bullet points describing your changes, use full sentences and try to link lines in the code whenever needed)
server conf changes

Details (add details if your pull request is more complicated and harder to understand from the code alone)
url updated for prod

Standard Qs (leave questions that do not apply blank)

If you broke behavior: Please describe what behavior you broke and how you inform people to not get stuck trying to use the old behavior.

If you used new dependencies: Did you add them to requirements.txt?
NA

Who did you ping on Mattermost to review your PR? Please ping that person again whenever you are ready for another review.
Sam and Liam


Please do not mark comments/conversations as resolved unless you are the assigned reviewer. This helps maintain clarity during the review process.

str issue on prod
Copy link
Collaborator

@SamuelGabriel SamuelGabriel left a comment

Choose a reason for hiding this comment

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

LGTM

@anshulg954 anshulg954 merged commit a845e04 into main May 17, 2024
1 check passed
@anshulg954 anshulg954 deleted the anshulg954-patch-1 branch May 17, 2024 13:18
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.

2 participants