-
Notifications
You must be signed in to change notification settings - Fork 18
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
Web 3474 #355
Web 3474 #355
Conversation
Caution Review failedThe pull request is closed. WalkthroughThe pull request introduces the new version [1.12.1] documented in the CHANGELOG, along with a package version bump. It adds support for custom HTML and JSON handling in web native display campaigns by introducing new message types and functions. The changes refine offline state management, enhance error checking regarding ACCOUNT_ID in multiple modules (events, notifications, profile), and improve the visual builder with dynamic CSS updates and element insertion. Additional constants have been added for managing UI elements and session information. Changes
Sequence Diagram(s)sequenceDiagram
participant Browser
participant CT_SDK as CleverTap SDK
participant NativeDisplay as nativeDisplay Module
participant DOM as Document Object Model
Browser->>CT_SDK: Receives message (type 5 or 6)
CT_SDK-->>NativeDisplay: Call renderCustomHtml() [if type 5]
CT_SDK-->>NativeDisplay: Call handleJson() [if type 6]
NativeDisplay->>DOM: Attempt to locate target element
NativeDisplay->>DOM: Replace outer HTML / dispatch event (raiseViewed/CT_web_native_display_json)
DOM-->>NativeDisplay: Acknowledgment of update
sequenceDiagram
participant User
participant VisualBuilder as renderVisualBuilder()
participant Process as processElement & addNewEl
participant DOM as Document Object Model
User->>VisualBuilder: Initiate visual builder render
VisualBuilder->>Process: Process element (check click tracking, CSS update)
Process->>DOM: Query and update element (using updateElementCSS)
Process->>Process: Use addNewEl for dynamic element insertion (with findSiblingSelector)
DOM-->>VisualBuilder: DOM updated with new elements
Possibly related PRs
Suggested reviewers
Poem
Tip 🌐 Web search-backed reviews and chat
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (15)
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Changes
Describe the key changes in this PR with the Jira Issue reference
Changes to Public Facing API if any
Please list the impact on the public facing API if any
How Has This Been Tested?
Describe the testing approach and any relevant configurations (e.g., environment, platform)
Checklist
Summary by CodeRabbit
New Features
Bug Fixes
Improvements