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

Disable xray in the PR review environment #1952

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

andrewleith
Copy link
Member

Summary | Résumé

Pr to disable xray in the PR review environments.

Test instructions | Instructions pour tester la modification

  • PR review env should work

@andrewleith andrewleith changed the title fix: pass the PR review env an ENV VAR (AWS_XRAY_SDK_ENABLED) to te… Disable xray in the PR review environment Sep 19, 2024
Copy link
Contributor

@P0NDER0SA P0NDER0SA left a comment

Choose a reason for hiding this comment

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

let's do this!

Copy link

@andrewleith andrewleith merged commit 75050b7 into main Sep 19, 2024
10 checks passed
@andrewleith andrewleith deleted the task/disable-xray-pr-review branch September 19, 2024 17:34
andrewleith added a commit that referenced this pull request Sep 23, 2024
* fix(templates): update the preview logic so it correctly translates the template category value and passes it to the API when saving

* fix(tests): update preview tests

* feat(ui tests): add new scenarios to cover preview path

* fix(tests): handle both when `FF_TEMPLATE_CATEGORY` is on and off

* fix(create template): pass the template_category_id when previewing while creating a new template

* task: try to disable xray in the PR review env

* temporary: remove XRAY code to see if review env works

* experiment: try using single quotes to init xray

* task: disable xray in DEV/TEST

* task: remove xray

* task: re-enable xray

* chore: formatting

* task: disable xray again

* task: undo things related to xray. fixed it in another PR (#1952)

* test(template_categories): add scenario for creating + previewing a template before saving

* chore: formatting

* fix(tests): have the ui test remove the template it creates

---------

Co-authored-by: Jumana B <[email protected]>
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