Skip to content

Commit d1f5a34

Browse files
committed
chore: update salt and add dev notes
1 parent ea691b7 commit d1f5a34

1 file changed

Lines changed: 14 additions & 1 deletion

File tree

mailing_list/views.py

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323

2424
logger = logging.getLogger(__name__)
2525

26-
_CONFIRM_SALT = "mailing-list-confirm"
26+
_CONFIRM_SALT = "mailing-list-confirm-a40b24dc-a26d-49ca-81d1-5b2fccb5fd7b"
2727
_CONFIRM_MAX_AGE = 7 * 24 * 60 * 60 # 7 days in seconds
2828

2929

@@ -68,6 +68,12 @@ def _send_confirmation_email(
6868
)
6969

7070

71+
"""
72+
This view is currently only being used in the demo page. This will later have to be adapted to be reused
73+
in the multi-selection modals.
74+
"""
75+
76+
7177
class SubscribeView(LoginRequiredMixin, View):
7278
def post(self, request):
7379
email = request.POST.get("email", "").strip()
@@ -147,6 +153,13 @@ def post(self, request):
147153
)
148154

149155

156+
"""
157+
This will be partially deprecated once multi-selection modals are implemented.
158+
Although multi-selection modals will take care of the sub/unsub logic, this view still shows how you'd
159+
want to set the _mailing_list_card.html state to show the 'pending' or 'subscribed' status.
160+
"""
161+
162+
150163
class QuickSubscribeView(View):
151164
"""Subscribe to a single list. Works for both authenticated and anonymous users.
152165

0 commit comments

Comments
 (0)