Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[NT-1503, NT-1505] Remove Creator Details and Project Summary Experiments #1288

Merged
merged 5 commits into from
Sep 11, 2020

Conversation

justinswart
Copy link
Contributor

📲 What

Removes the native_me_project_summary and native_project_page_conversion_creator_details A/B experiments that we were running with Optimizely.

🤔 Why

We're no longer running these experiments and have reverted to the control experience.

🛠 How

  • Removed the experiments.
  • Removed any code affected by these experiments, including multiple network requests that we were making on the project page.
  • Fixed an unrelated issue in which our Library framework had our Kickstarter_Framework linked as a library.
  • Fixed a bunch of target memberships.

👀 See

Original PR's for these experiments:

#1093, #1148

✅ Acceptance criteria

  • There should be no regressions on the project page.

@nativeksr
Copy link
Collaborator

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

@justinswart justinswart merged commit ec4892c into master Sep 11, 2020
@justinswart justinswart deleted the NT-1503-remove-project-summary-experiment branch September 11, 2020 17:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants