Skip to content

feat: add search_default_sort_newest waffle flag (ENT-11384)#2638

Draft
macdiesel wants to merge 1 commit into
openedx:masterfrom
macdiesel:bbeggs/search-default-sort-newest-flag
Draft

feat: add search_default_sort_newest waffle flag (ENT-11384)#2638
macdiesel wants to merge 1 commit into
openedx:masterfrom
macdiesel:bbeggs/search-default-sort-newest-flag

Conversation

@macdiesel

Copy link
Copy Markdown
Contributor

What

Adds the enterprise.search_default_sort_newest waffle flag, surfaced via enterprise_features, to gate the Learner Portal "newest courses first" search sort. This is the eligibility gate / kill-switch for the Optimizely A/B experiment.

  • SEARCH_DEFAULT_SORT_NEWEST WaffleFlag + search_default_sort_newest_enabled() helper + entry in enterprise_features() (enterprise/toggles.py).
  • Updated the two exact-match enterprise_features test fixtures (test_views.py, test_filters.py).
  • Version bump 7.0.0 → 7.0.1 + CHANGELOG entry.

Testing

The two affected tests pass in an isolated venv (pinned Django 5.2.12): TestEnterpriseCustomerViewSet::test_enterprise_customer_with_access_to (15 passed) and TestEnterpriseLinkedUserFilterBackend::test_filter (4 passed).

Related (rollout order)

  1. enterprise-catalog recency replica → feat: "newest courses first" Algolia sort — config-gated replica registry + fail-safe edx/enterprise-catalog#118
  2. this PR (waffle flag)
  3. frontend-app-learner-portal-enterprise index swap + Optimizely experiment

ENT-11384.

🤖 Generated with Claude Code

Adds the enterprise.search_default_sort_newest WaffleFlag, surfaced via enterprise_features, to gate the Learner Portal "newest courses first" search sort (eligibility gate / kill-switch for the Optimizely A/B experiment). Bumps version to 7.0.1 and updates the two exact-match enterprise_features test fixtures.

ENT-11384

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@iloveagent57

Copy link
Copy Markdown
Contributor

you appear to be quite far behind master, though

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