-
Notifications
You must be signed in to change notification settings - Fork 9
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
[MOB-8539]: Embedded GA #381
Merged
Merged
Conversation
This file contains 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
* notificiation view implemented * code changes as per recent review * usage of banner view * usage of notification view * Webpack config changed for Notification View --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com>
* Embedded Manager Module - WIP * Embedded Manager Module - added uuid library * Code modified for embedded msg * Documentation added * Events example added * Embedded msg configurations changed * Import issue of embeddedManager fixed * Circuler dependency solved * Linting error fixing * events code break down into separate folders * Import issues fixed --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com>
* Embedded Manager Module - WIP * Embedded Manager Module - added uuid library * Code modified for embedded msg * Documentation added * Events example added * Test Cases added for embedded message manager * Added Embedded Placement Test cases * EmbeddedMsg Test Cases added * embeddedManager import issue fixed * Linting error resolved * Test issues resolved * basic changes to run the code * change query params of get embedded messaging api * remove code for embedded messaging sync parameters * revert the changes that were made earlier to run the code. --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: Hani <haniramani@gmail.com>
* card view implemented * code changes as per recent review * Embedded Manager Module - WIP * Embedded Manager Module - added uuid library * usage of card view * Code modified for embedded msg * Documentation added * Events example added * Embedded msg example with card view * Embedded msg example updated * build config changed and few updates done on card component * react-example improved * Node webpack config changed * Webpack config changed * Yarn file issue fixed * Resolve revice comments --------- Co-authored-by: iamavishkar <imavishkar@gmail.com> Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: Hani <haniramani@gmail.com>
* set embedded message dismiss event * remove unnecessary file * display API response to the embedded page * Resolve review comments --------- Co-authored-by: Hani <haniramani@gmail.com>
* Embedded Manager Module - WIP * Embedded Manager Module - added uuid library * Code modified for embedded msg * Documentation added * Events example added * Test Cases added for embedded message manager * Added Embedded Placement Test cases * EmbeddedMsg Test Cases added * embeddedManager import issue fixed * Linting error resolved * Test issues resolved * basic changes to run the code * change query params of get embedded messaging api * change placementIds param format in API query param * change query param in sample app * make placement id param optional * modify test case accordingly --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: Hani <haniramani@gmail.com>
* banner view implemented * code changes as per recent review * usage of banner view * Banner View Webpack config changed * .babelrc config changed * [MOB-7175]: add new filter method that leaves in JSON only messages (#238) * add new filter method that leaves in JSON only messages * Mentioning filterOnlyReadAndNeverTriggerMessages --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Brad Umbaugh <brad.umbaugh@iterable.com> * ver bump (#240) Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: devcsomnicg <129495456+devcsomnicg@users.noreply.github.com> Co-authored-by: Mitch Prewitt <44011584+mprew97@users.noreply.github.com> Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Brad Umbaugh <brad.umbaugh@iterable.com> Co-authored-by: Hani <haniramani@gmail.com>
* card view implemented * code changes as per recent review * usage of card view * build config changed and few updates done on card component * react-example improved * Node webpack config changed * Webpack config changed --------- Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: Hani <haniramani@gmail.com> Co-authored-by: Mitch Prewitt <44011584+mprew97@users.noreply.github.com>
* card view implemented * code changes as per recent review * Embedded Manager Module - WIP * Embedded Manager Module - added uuid library * usage of card view * Code modified for embedded msg * Documentation added * Events example added * Embedded msg example with card view * Embedded msg example updated * build config changed and few updates done on card component * react-example improved * Create manual.yml * Node webpack config changed * Webpack config changed * Yarn file issue fixed * [MOB-7175]: add new filter method that leaves in JSON only messages (#238) * add new filter method that leaves in JSON only messages * Mentioning filterOnlyReadAndNeverTriggerMessages --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Brad Umbaugh <brad.umbaugh@iterable.com> * ver bump (#240) Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> * embedded msg view integration * integrate demo in sample app * Resolve node build issue * commit remaining code * commit remaining files --------- Co-authored-by: iamavishkar <imavishkar@gmail.com> Co-authored-by: Vishal Joshi <joshivishal895@gmail.com> Co-authored-by: Mitch Prewitt <44011584+mprew97@users.noreply.github.com> Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Brad Umbaugh <brad.umbaugh@iterable.com> Co-authored-by: Hani <haniramani@gmail.com>
This reverts commit 3704505.
* Add remaining events to web embedded messaging * Resolve review comments --------- Co-authored-by: Hani Vora <hani@Hanis-MacBook-Air.local>
* Create test (#250) * Create test * Update yarn.lock (#251) * Update events.schema.ts (#257) * Update types.ts (#262) * Update events.schema.ts (#259) * Update EmbeddedForm.tsx (#253) * Delete test (#254) * Update EmbeddedMsgs.tsx (#255) * Update embeddedManager.ts (#256) * Update types.ts (#258) * Update events.test.ts (#260) * Update events.ts (#261) * Update EmbeddedForm.tsx (#290) * Create Functions.ts (#291) * Update embeddedManager.ts (#292) * Update events.ts (#293) * Create functions (#294) * Rename functions to functions.ts (#298) * Update embeddedManager.test.ts (#299)
* change in embedded api params * Update EmbeddedMsgs.tsx (#263) * Update index.tsx (#264) * Update index.tsx (#265) * Update index.tsx (#266) * Update embeddedManager.ts (#267) * Update index.ts (#268) * Update events.schema.ts (#270) * Update types.ts (#271) * Update yarn.lock (#272) * Create IterableActionRunner.ts (#274) * Update embeddedManager.ts (#276) * Update types.ts (#277) * Resolve build fail issue (#289) * Resolve merge conflicts
* Create test (#250) * Create test * Update yarn.lock (#251) * Update events.schema.ts (#257) * Update types.ts (#262) * Update events.schema.ts (#259) * Update EmbeddedForm.tsx (#253) * Delete test (#254) * Update EmbeddedMsgs.tsx (#255) * Update embeddedManager.ts (#256) * Update types.ts (#258) * Update events.test.ts (#260) * Update events.ts (#261) * Update index.tsx (#284) * Update index.tsx (#285) * Resolve merge conflicts
* Create test (#250) * Create test * Update yarn.lock (#251) * Update events.schema.ts (#257) * Update types.ts (#262) * Update events.schema.ts (#259) * Update EmbeddedForm.tsx (#253) * Delete test (#254) * Update EmbeddedMsgs.tsx (#255) * Update embeddedManager.ts (#256) * Update types.ts (#258) * Update events.test.ts (#260) * Update events.ts (#261) * Update index.tsx (#279) * Update index.tsx (#280) * Update index.ts (#281) * Update index.tsx (#282) * Update index.tsx (#295) * Update index.tsx (#296) * Update index.tsx (#297)
* change in embedded api params * Update EmbeddedMsgs.tsx (#263) * Update index.tsx (#264) * Update index.tsx (#265) * Update index.tsx (#266) * Update embeddedManager.ts (#267) * Update index.ts (#268) * Update events.schema.ts (#270) * Update types.ts (#271) * Update yarn.lock (#272) * Create IterableActionRunner.ts (#274) * Remove unused code (#275) * Update embeddedManager.ts (#287)
* Make UI responsive * Resolve review comments
* set message if message is not available * resolve review comments
* modify button UI * resolve merge conflicts
* fix * fix --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* fix publish issue * revert lock changes * fix * now? --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * fixed client's comments * Fixed comments * updated click events as common * HTTP call in retrieveEmbeddedMessages (#355) * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * HTTP call in retrieveEmbeddedMessages * updated test file * fixed client's comments * Fixed comments * updated click events as common * sdk and platform updated * updated test file * Web SDK is incorrectly parsing custom action objects * updated listener name and handle callback * Updated custom action object type * HTTP call in retrieveEmbeddedMessages (#355) * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * Too many Embedded-related interfaces? (#359) * Updated interfaces * removed duplicate interface from in-app * Updated IEmbeddedMessage in events * Updated EnbeddedMessagingDismiss, EnbeddedMessagingSession * Removed unused interfaces * Removed unused interfaces * Updated test.ts --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * updated code in useeffect * pass package name and expose trackembeddedclick as standalone function (#363) --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * HTTP call in retrieveEmbeddedMessages * updated test file * fixed client's comments * Fixed comments * updated click events as common * sdk and platform updated * updated test file * Web SDK is incorrectly parsing custom action objects * updated listener name and handle callback * Updated custom action object type * HTTP call in retrieveEmbeddedMessages (#355) * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * Too many Embedded-related interfaces? (#359) * Updated interfaces * removed duplicate interface from in-app * Updated IEmbeddedMessage in events * Updated EnbeddedMessagingDismiss, EnbeddedMessagingSession * Removed unused interfaces * Removed unused interfaces * Updated test.ts --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * updated code in useeffect * pass package name and expose trackembeddedclick as standalone function * Update Embedded requests to not need to pass userId or email * removed unused code from test * Updated endpoints in setUserId and setEmail * added impression endpoint --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file * Web SDK is incorrectly parsing custom action objects * updated listener name and handle callback * Updated custom action object type * Too many Embedded-related interfaces? (#359) * Updated interfaces * removed duplicate interface from in-app * Updated IEmbeddedMessage in events * Updated EnbeddedMessagingDismiss, EnbeddedMessagingSession * Removed unused interfaces * Removed unused interfaces * Updated test.ts --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * Rework Embedded Session Manager * Updated messageId * Update types.ts * Update EmbeddedMsgs.tsx * Fixed end session event in SDK --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file * Web SDK is incorrectly parsing custom action objects * updated listener name and handle callback * Updated custom action object type * Too many Embedded-related interfaces? (#359) * Updated interfaces * removed duplicate interface from in-app * Updated IEmbeddedMessage in events * Updated EnbeddedMessagingDismiss, EnbeddedMessagingSession * Removed unused interfaces * Removed unused interfaces * Updated test.ts --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * Rework Embedded Session Manager * Updated messageId * Update types.ts * Update EmbeddedMsgs.tsx * remove react dep * Fixed end session event in SDK * checking * need to see * remove dupe schema * cleanup * clean up packages and more * typo * fix tests and change requests * further change requests * oop * fix circular deps * additional cleanup and rework * fix embeddedManager test * fix base branch issues * rename to iterable * update OOTB names * export --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> Co-authored-by: hardikmashru <150107929+hardikmashru@users.noreply.github.com> Co-authored-by: Hardik Mashru <hardik.mashru@iterable.com> Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* OOTB Rework: Need to be pure JS/HTML/CSS * OOTB Rework: Need to be pure JS/HTML/CSS * added id of all component * added custom changes * fixed click issue * fixed comments * handled click by elements name * updated example * fixed cursor pointer * Update EmbeddedMsgs.tsx (sample code) to use EmbeddedMessageUpdateHandler * HTTP call in retrieveEmbeddedMessages * updated test file * sdk and platform updated * updated test file * Web SDK is incorrectly parsing custom action objects * updated listener name and handle callback * Updated custom action object type * Too many Embedded-related interfaces? (#359) * Updated interfaces * removed duplicate interface from in-app * Updated IEmbeddedMessage in events * Updated EnbeddedMessagingDismiss, EnbeddedMessagingSession * Removed unused interfaces * Removed unused interfaces * Updated test.ts --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> * Rework Embedded Session Manager * Updated messageId * Update types.ts * Update EmbeddedMsgs.tsx * remove react dep * Fixed end session event in SDK * checking * need to see * remove dupe schema * cleanup * clean up packages and more * typo * fix tests and change requests * further change requests * oop * fix circular deps * additional cleanup and rework * fix embeddedManager test * fix base branch issues * update lint rules * more tweaks --------- Co-authored-by: hardikmashru <hardikmashru@iterable.com> Co-authored-by: hardikmashru <150107929+hardikmashru@users.noreply.github.com> Co-authored-by: Hardik Mashru <hardik.mashru@iterable.com> Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* pushed changes * Error handling for trackEmbeddedClick Error handling for trackEmbeddedClick * sample app changes * revert unnecessary react-example changes * revert react-example unnecessary changes * implemented changes as per review suggestions * Update request.ts * create utility method for getting url from action (#371) create utility method for getting url from action
* easier UUID testing in sample app * some cleanup and change request additions --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* easier UUID testing in sample app * hopping * update interceptors and routes * oops * accidental delete from merge * forgot route * wrong route * oops * Update src/constants.ts Co-authored-by: Paul Jung <paul.jung@iterable.com> * update const name --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Paul Jung <paul.jung@iterable.com>
* easier UUID testing in sample app * hopping * update interceptors and routes * oops * accidental delete from merge * forgot route * wrong route * oops * Update src/constants.ts Co-authored-by: Paul Jung <paul.jung@iterable.com> * update const name * remove timeout --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com> Co-authored-by: Paul Jung <paul.jung@iterable.com>
* pushed changes * Update LoginForm.tsx * fix cardView, notificationView, bannerView * fix aspect-ratio, padding * fix margin-top for banner image * revert LoginForm changes * final fix card, banner, notification view * UI fixes, responsive fixes and impression tracker screen fixes * put back env.example * rename env.example * removed inline style * rollback for unnecessary PR diff * Update index.tsx * update .evn.example * fix card, banner, notification view * updates * escape htmltext --------- Co-authored-by: Hardik Mashru <harrymash2006@gmail.com>
* mostly up to date * updates * teeny * button cleanup * oop * add style update examples * jsdoc comments --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* update trackNewlyRetrieved * clean up --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* expose handleEmbeddedClick * fix embeddedClick alias * Update utils.ts * Update utils.ts * rename --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* add elements to OOTB type * update var name --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
* update to target url * update func name --------- Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
Co-authored-by: mitch prewitt <mitch.prewitt@iterable.com>
jyu115
approved these changes
Jun 11, 2024
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.
JIRA Ticket(s) if any
Description
For Embedded GA release coming soon.
Test Steps