feat: consent notice on sign-in, and interactive messages people can answer - #6
Open
rvignesh89 wants to merge 3 commits into
Open
feat: consent notice on sign-in, and interactive messages people can answer#6rvignesh89 wants to merge 3 commits into
rvignesh89 wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
rvignesh89
force-pushed
the
web-channel-optin-admin-send
branch
from
September 6, 2026 21:34
b63341c to
1a06726
Compare
rvignesh89
force-pushed
the
web-channel-optin-admin-send
branch
from
September 7, 2026 15:07
0b7b57e to
617b4f2
Compare
Glific no longer opts a web login in to WhatsApp (#5713); signing in records consent for the web channel alone. The notice sits on the phone-entry step, before the number is submitted, because signing in is what records that consent — putting it on the OTP step would ask after the fact. A notice rather than a checkbox: consent is implied by continuing. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015JqHf9PqoGCW27fcNbQnnT
Staff can now send an interactive template to a web contact (glific#5717), but the widget rendered it as its plain body — the options were invisible and untappable, so the feature did not work end to end. Quick replies show their options directly. A list's options live one level down inside sections; they are flattened into the same buttons, since the widget has no room for WhatsApp's two-step "open the list, then choose". Tapping an option answers with its title as an ordinary text message, which is exactly what WhatsApp records for an interactive reply — so nothing downstream, the message row, the staff inbox, a future flow, has to know it came from a tap. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015JqHf9PqoGCW27fcNbQnnT
The notice was written against the org-name endpoint this branch used before main replaced it with runtime branding. After the rebase `orgName` no longer existed and every Login test failed on a ReferenceError. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015JqHf9PqoGCW27fcNbQnnT
AmishaBisht
force-pushed
the
web-channel-optin-admin-send
branch
from
September 8, 2026 17:19
617b4f2 to
7cad147
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of glific/glific#5713 and epic glific/glific#5659.
Stacked on
web-channel-send-messages(#5), notmain. Backend half: glific/glific#5717.The consent notice (#5713)
Glific no longer opts a web login in to WhatsApp; signing in records consent for the web channel alone, in a new per-channel table. The widget has to say so.
The notice sits on the phone-entry step, above the submit button — signing in is what records the consent, so putting it on the OTP step would ask after the number has already been submitted. It names the organisation, taken from the name the widget already fetches.
A notice rather than a checkbox: consent is implied by continuing, which is the product requirement.
Interactive messages
glific#5717 lets staff send an interactive template to a web contact. Without this the widget rendered it as its plain body: the options were invisible and untappable, so the feature did not actually work end to end.
Staff replies themselves needed no change here: the widget already listens for
new_messageon the contact's topic and rendersflow: "outbound"on the left, so an admin reply appears live with nothing added.Tests
167 passing. New coverage for the notice (present on the phone step, absent on the OTP step — the second is what catches someone moving it later), and for interactive rendering: quick-reply options, list flattening including a section option's description, the title reported on tap, and no buttons on the contact's own message.
🤖 Generated with Claude Code
https://claude.ai/code/session_015JqHf9PqoGCW27fcNbQnnT