Skip to content

Add favorites list plugin to application defaults #2322

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

Merged
merged 1 commit into from
Jun 30, 2025

Conversation

shanbady
Copy link
Contributor

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/7544

Description (What does it do?)

This PR adds the FavoritesListPlugin plugin to application defaults. FavoritesListPlugin is responsible for automatically creating a placeholder favorites list for users upon signup.

How can this be tested?

NOTE: this already works locally. this fix is intended for deployed environments

  1. checkout this branch
  2. create a new user or login as existing
  3. post-auth check to see that they have at least one favorites list via:
from django.contrib.auth import get_user_model
User = get_user_model()
user = User.objects.get(id=<your user id>)
assert user.user_lists.count() >= 1

@shanbady shanbady added the Needs Review An open Pull Request that is ready for review label Jun 27, 2025
@shanbady shanbady marked this pull request as ready for review June 27, 2025 13:38
@mbertrand mbertrand self-assigned this Jun 30, 2025
Copy link
Member

@mbertrand mbertrand left a comment

Choose a reason for hiding this comment

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

👍

@mbertrand mbertrand added Waiting on author and removed Needs Review An open Pull Request that is ready for review labels Jun 30, 2025
@shanbady shanbady merged commit ab016c4 into main Jun 30, 2025
13 checks passed
@shanbady shanbady deleted the shanbady/fix-auth-plugin-defaults branch June 30, 2025 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants