Skip to content

Remove EXPORT_DATA_SOURCE_DATA feature flag #36664

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kaapstorm
Copy link
Contributor

@kaapstorm kaapstorm commented Jul 4, 2025

Product Description

Removes the "Add Export Data Source Data page" feature flag, and underlying functionality.

This feature allowed users to export data sources the same way that they can export forms and cases. In the past this was used to copy data sources to remote databases using the Data Export Tool.

Part of the "GA or Bust" strategic initiative. No live domains are using this.

Technical Summary

Jira: SAAS-17785

Feature Flag

EXPORT_DATA_SOURCE_DATA

Safety Assurance

Safety story

Used by projects whose subscriptions are not active.

Automated test coverage

Test coverage removed

QA Plan

No QA plan

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@kaapstorm kaapstorm requested review from mkangia and Charl1996 July 4, 2025 12:01
@kaapstorm kaapstorm added Open for review: do not merge A work in progress product/feature-flag Change will only affect users who have a specific feature flag enabled labels Jul 4, 2025
@dimagimon dimagimon added the Risk: High Change affects files that have been flagged as high risk. label Jul 4, 2025
@mkangia
Copy link
Contributor

mkangia commented Jul 7, 2025

Heads up that you might want to hold off on merging this for a bit
https://dimagi.atlassian.net/browse/SAAS-17786?focusedCommentId=386629

Copy link
Contributor

@mkangia mkangia left a comment

Choose a reason for hiding this comment

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

Quite the clean up!
Changes look good. Got some test failure though.

Copy link
Contributor

@mkangia mkangia left a comment

Choose a reason for hiding this comment

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

+5 −573

And it begins!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Open for review: do not merge A work in progress product/feature-flag Change will only affect users who have a specific feature flag enabled Risk: High Change affects files that have been flagged as high risk.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants