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.

Sign up for GitHub

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

Jump to bottom

CORE-201 Shut down Community App - Beta,Staging,Test & QA envs #6957

Merged
merged 11 commits into from
Jan 19, 2024
Merged

CORE-201 Shut down Community App - Beta,Staging,Test & QA envs #6957

merged 11 commits into from
Jan 19, 2024

Conversation

kkartunov
Copy link
Collaborator

@kkartunov kkartunov commented Jan 17, 2024 •
edited
Loading

@kkartunov kkartunov requested review from jmgasper and removed request for ajefts and luizrrodrigues January 17, 2024 07:28
jmgasper
jmgasper approved these changes Jan 17, 2024
View reviewed changes
Copy link
Collaborator

@jmgasper jmgasper left a comment

Choose a reason for hiding this comment

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

Looks good. I don't see anything that will break staging, but just wanted to make sure this won't affect staging, since we still use that

@kkartunov kkartunov changed the title CORE-201 Shut down Community App- Beta CORE-201 Shut down Community App - Beta & Staging Jan 17, 2024
@kkartunov
Copy link
Collaborator Author

@jmgasper actually we are extending this for staging as well.

@jmgasper
Copy link
Collaborator

@jmgasper actually we are extending this for staging as well.

We use staging for QA validation of prod fixes. We can work around not having it, but it is useful at times.

@kkartunov kkartunov requested a review from testflyjets January 17, 2024 08:31
@kkartunov
Copy link
Collaborator Author

@testflyjets FYI.

testflyjets
testflyjets approved these changes Jan 17, 2024
View reviewed changes
Copy link

@testflyjets testflyjets left a comment

Choose a reason for hiding this comment

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

Please add a comment at the top of the README saying this code has been DEPRECATED as of today.

@kkartunov kkartunov changed the title CORE-201 Shut down Community App - Beta & Staging CORE-201 Shut down Community App - Beta,Staging,Test & QA envs Jan 19, 2024
@kkartunov kkartunov merged commit 270777a into master Jan 19, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@testflyjets testflyjets testflyjets approved these changes

@jmgasper jmgasper jmgasper approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kkartunov @jmgasper @testflyjets