Open
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a comprehensive integration with Creator LMS, enabling users to automate various learning management system tasks directly from the application. It includes both backend logic for API interactions and a complete frontend user interface for seamless configuration, enhancing the platform's extensibility and automation capabilities for educational content management. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Pull request overview
Adds a new Creator LMS integration across the frontend and backend, wiring it into the integrations UI and exposing a backend authorize endpoint plus execution plumbing.
Changes:
- Added Creator LMS frontend integration screens (authorization, action selection, field mapping, edit flow) and supporting static field/action metadata.
- Registered Creator LMS in integration selectors/routing (new action selection, edit, info views, tutorial link, webhook/static integration lists).
- Added backend Creator LMS action route, controller, and record execution helper; added Creator LMS to the backend trigger/action name registry.
Reviewed changes
Copilot reviewed 28 out of 29 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/resource/img/integ/creatorLms.webp | Adds Creator LMS integration icon asset. |
| frontend/src/components/Flow/New/SelectAction.jsx | Adds CreatorLms to the “select action” list. |
| frontend/src/components/AllIntegrations/NewInteg.jsx | Lazy-loads and routes to the new CreatorLms integration component. |
| frontend/src/components/AllIntegrations/IntegInfo.jsx | Adds CreatorLms authorization/info rendering case. |
| frontend/src/components/AllIntegrations/EditInteg.jsx | Adds CreatorLms edit routing. |
| frontend/src/components/AllIntegrations/CreatorLms/* | Implements Creator LMS integration UI (authorize, map fields, edit, static data). |
| frontend/src/Utils/StaticData/tutorialLinks.js | Adds Creator LMS documentation link. |
| frontend/src/Utils/StaticData/webhookIntegrations.js | Adds CreatorLms to the customFormIntegrations list. |
| frontend/src/components/AllIntegrations//.jsx (several) | Raises step-2 minHeight from 200px to 500px in multiple integration UIs. |
| backend/Core/Util/AllTriggersName.php | Registers CreatorLms in the backend integration registry. |
| backend/Actions/CreatorLms/Routes.php | Adds creator_lms_authorize POST route. |
| backend/Actions/CreatorLms/CreatorLmsController.php | Implements authorize handler and execution entrypoint. |
| backend/Actions/CreatorLms/RecordApiHelper.php | Implements main-action dispatch + logging for Creator LMS actions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsAuthorization.jsx
Show resolved
Hide resolved
frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js
Outdated
Show resolved
Hide resolved
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new integration for Creator LMS. The changes include new backend controllers, API helpers, and routes, as well as new frontend components for setting up and editing the integration. The implementation is well-structured and follows the existing patterns in the codebase. I've provided a few suggestions to improve code quality, including fixing a typo, removing redundant code, and adhering to best practices in both PHP and React.
frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Rishad Alam <sazzadullalamrishad@gmail.com>
There was a problem hiding this comment.
Pull request overview
Adds a new Creator LMS action integration across the frontend integration UI and backend action execution plumbing, plus some related UI sizing and CI build workflow updates.
Changes:
- Introduces Creator LMS integration UI (authorization, action selection, field mapping, edit/info support) and adds its logo/tutorial link.
- Adds backend Creator LMS action route/controller/helper to authorize and execute mapped actions via hooks.
- Updates several integration step-2 containers to use a larger
minHeightand updates plugin-check workflow to use.github/build.
Reviewed changes
Copilot reviewed 29 out of 30 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/resource/img/integ/creatorLms.webp | Adds Creator LMS integration logo asset. |
| frontend/src/components/Flow/New/SelectAction.jsx | Adds CreatorLms to action selection list. |
| frontend/src/components/AllIntegrations/NewInteg.jsx | Lazy-loads and routes to the new Creator LMS integration component. |
| frontend/src/components/AllIntegrations/IntegInfo.jsx | Adds Creator LMS info/authorization view wiring. |
| frontend/src/components/AllIntegrations/EditInteg.jsx | Adds Creator LMS edit integration wiring. |
| frontend/src/components/AllIntegrations/CreatorLms/staticData.js | Defines Creator LMS modules and field definitions/options. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLms.jsx | Implements the 3-step Creator LMS integration setup flow. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsAuthorization.jsx | Implements “connect/authorize” step for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx | Implements action selection + field mapping UI for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsFieldMap.jsx | Implements per-row field mapping UI for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js | Shared helpers for input handling + mapped-field validation. |
| frontend/src/components/AllIntegrations/CreatorLms/EditCreatorLms.jsx | Implements edit screen for existing Creator LMS configs. |
| frontend/src/Utils/StaticData/tutorialLinks.js | Adds Creator LMS documentation link entry. |
| frontend/src/Utils/StaticData/webhookIntegrations.js | Adds CreatorLms to static integration lists. |
| backend/Actions/CreatorLms/Routes.php | Adds backend AJAX route for Creator LMS authorization. |
| backend/Actions/CreatorLms/CreatorLmsController.php | Adds authorization check and execution entrypoint for Creator LMS actions. |
| backend/Actions/CreatorLms/RecordApiHelper.php | Maps field data and dispatches action execution via hooks + logging. |
| backend/Core/Util/AllTriggersName.php | Adds Creator LMS label entry to trigger-name list (currently problematic). |
| frontend/src/components/AllIntegrations/Autonami/Autonami.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/FluentCRM/FluentCrm.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/FluentCart/FluentCart.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/MailPoet/MailPoet.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/MailerPress/MailerPress.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/SeoPress/SeoPress.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/TeamsForWooCommerceMemberships/TeamsForWooCommerceMemberships.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/Telegram/Telegram.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/UserRegistrationMembership/UserRegistrationMembership.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/WPCafe/WPCafe.jsx | Updates step-2 container minHeight to 500px. |
| frontend/src/components/AllIntegrations/WPCourseware/WPCourseware.jsx | Updates step-2 container minHeight to 500px (two places). |
| .github/workflows/plugin-check.yml | Switches build packaging step to bash .github/build. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx
Show resolved
Hide resolved
🔍 WordPress Plugin Check Report
📊 Report
|
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
mismatched_plugin_name | Plugin name "Bit integrations - Easy Automator with no-code automation, integrate Webhook and automate 300+ Platform" is different from the name declared in plugin header "Bit Integrations". |
0 |
readme_parser_warnings_trimmed_section_description | The "Description" section is too long and was truncated. A maximum of 2500 characters is supported. |
🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check
There was a problem hiding this comment.
Pull request overview
Adds a new Creator LMS action integration across the frontend flow builder and backend action handler scaffolding, plus a small CI/build workflow adjustment and some step-2 UI height tweaks for multiple integrations.
Changes:
- Added Creator LMS integration UI (authorization, action selection, field mapping, edit/info views) and its logo asset.
- Added backend Creator LMS action controller/route and execution helper to dispatch actions via integration hooks.
- Updated plugin-check workflow build step and increased several integrations’ step-2
minHeight.
Reviewed changes
Copilot reviewed 29 out of 30 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/resource/img/integ/creatorLms.webp | Adds Creator LMS logo asset for integration lists/cards. |
| frontend/src/components/Flow/New/SelectAction.jsx | Makes CreatorLms selectable as an action in the flow builder. |
| frontend/src/components/AllIntegrations/NewInteg.jsx | Wires CreatorLms into the “new integration” renderer/switch. |
| frontend/src/components/AllIntegrations/IntegInfo.jsx | Adds CreatorLms authorization/info rendering. |
| frontend/src/components/AllIntegrations/EditInteg.jsx | Adds CreatorLms edit screen routing. |
| frontend/src/components/AllIntegrations/CreatorLms/staticData.js | Defines Creator LMS actions/modules and field definitions. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLms.jsx | Implements Creator LMS step flow (auth → mapping → save). |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsAuthorization.jsx | Adds “check installed/active” authorization step for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx | Adds action selection, optional course status selection, and field map UI. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsFieldMap.jsx | Implements per-row mapping UI for Creator LMS fields. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js | Adds shared helpers for config input + mapping validation/generation. |
| frontend/src/components/AllIntegrations/CreatorLms/EditCreatorLms.jsx | Adds edit mode wrapper for Creator LMS configs. |
| frontend/src/Utils/StaticData/webhookIntegrations.js | Allows CreatorLms in customFormIntegrations list. |
| frontend/src/Utils/StaticData/tutorialLinks.js | Adds Creator LMS documentation link entry. |
| frontend/src/components/AllIntegrations/Autonami/Autonami.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/FluentCart/FluentCart.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/FluentCRM/FluentCrm.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/MailerPress/MailerPress.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/MailPoet/MailPoet.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/WPCafe/WPCafe.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/WPCourseware/WPCourseware.jsx | Increases step-2 container minHeight (two places). |
| frontend/src/components/AllIntegrations/SeoPress/SeoPress.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/Telegram/Telegram.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/TeamsForWooCommerceMemberships/TeamsForWooCommerceMemberships.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/UserRegistrationMembership/UserRegistrationMembership.jsx | Increases step-2 container minHeight. |
| backend/Core/Util/AllTriggersName.php | Registers CreatorLms in the backend integration name list. |
| backend/Actions/CreatorLms/Routes.php | Adds AJAX route for creator_lms_authorize. |
| backend/Actions/CreatorLms/CreatorLmsController.php | Adds authorization check + execution entrypoint for Creator LMS action. |
| backend/Actions/CreatorLms/RecordApiHelper.php | Adds action dispatch scaffolding (hook-based) + logging for Creator LMS. |
| .github/workflows/plugin-check.yml | Switches build script and modifies plugin-check checks list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
This PR adds a new Creator LMS action integration across the frontend and backend, including UI configuration screens, authorization endpoint wiring, and backend execution helpers. It also includes a set of UI layout tweaks (step-2 min height) across multiple integrations and updates the CI plugin-check build step.
Changes:
- Add Creator LMS integration UI (create/edit/info flows) and supporting static field/action definitions.
- Add backend Creator LMS action route/controller/record helper for authorization and execution via hooks.
- Adjust several integration step-2 containers’
minHeight, and update the plugin-check workflow/build script.
Reviewed changes
Copilot reviewed 31 out of 32 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/resource/img/integ/creatorLms.webp | Adds Creator LMS logo asset for integration cards. |
| frontend/src/components/Flow/New/SelectAction.jsx | Registers CreatorLms in the action selection list. |
| frontend/src/components/AllIntegrations/NewInteg.jsx | Lazy-loads and routes to the CreatorLms integration component. |
| frontend/src/components/AllIntegrations/IntegInfo.jsx | Adds CreatorLms info view (authorization component). |
| frontend/src/components/AllIntegrations/EditInteg.jsx | Adds CreatorLms edit screen routing/lazy-load. |
| frontend/src/components/AllIntegrations/CreatorLms/staticData.js | Defines Creator LMS modules and field schemas for mapping. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLms.jsx | New Creator LMS “create integration” multi-step UI. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsAuthorization.jsx | Authorization UI calling creator_lms_authorize. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx | Action selection + field mapping UI for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsFieldMap.jsx | Field-map row component for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js | Shared helpers for input + mapping validation/default mapping. |
| frontend/src/components/AllIntegrations/CreatorLms/EditCreatorLms.jsx | Edit flow for existing Creator LMS integrations. |
| frontend/src/components/AllIntegrations/WPCourseware/WPCourseware.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/WPCafe/WPCafe.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/UserRegistrationMembership/UserRegistrationMembership.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/Telegram/Telegram.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/TeamsForWooCommerceMemberships/TeamsForWooCommerceMemberships.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/SeoPress/SeoPress.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/MailerPress/MailerPress.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/MailPoet/MailPoet.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/FluentCart/FluentCart.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/FluentCRM/FluentCrm.jsx | Increases step-2 container minHeight. |
| frontend/src/components/AllIntegrations/Autonami/Autonami.jsx | Increases step-2 container minHeight. |
| frontend/src/Utils/StaticData/webhookIntegrations.js | Adds CreatorLms to customFormIntegrations list. |
| frontend/src/Utils/StaticData/tutorialLinks.js | Adds Creator LMS documentation link entry. |
| backend/Core/Util/AllTriggersName.php | Adds CreatorLms entry to pro trigger-name list. |
| backend/Actions/CreatorLms/Routes.php | Registers creator_lms_authorize AJAX route. |
| backend/Actions/CreatorLms/CreatorLmsController.php | Adds authorize handler + execute entrypoint for Creator LMS action. |
| backend/Actions/CreatorLms/RecordApiHelper.php | Adds hook-based execution + logging for Creator LMS actions. |
| backend/Actions/Keap/RecordApiHelper.php | Adjusts logging and tag insertion flow for Keap. |
| .github/workflows/plugin-check.yml | Uses .github/build and removes plugin_header check. |
| .github/build | Stops copying build-hash.txt into build directory. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+38
to
+40
| draftConf.mainAction = value | ||
| draftConf.courseStatus = null | ||
|
|
Comment on lines
145
to
+147
| if ($defaultConf->actions->tags || isset($apiResponse->id)) { | ||
| $tagResponse = $this->insertTag($apiResponse->id, $defaultConf->selectedTags); | ||
| } | ||
| $reponseType = HttpHelper::$responseCode === 200 ? 'success' : 'error'; |
Comment on lines
+147
to
+149
| $reponseType = HttpHelper::$responseCode === 200 ? 'success' : 'error'; | ||
|
|
||
| if (!(isset($apiResponse->id))) { | ||
| LogHandler::save($this->_integrationID, ['type' => 'contact', 'type_name' => 'add-contact'], 'error', $apiResponse); | ||
| } else { | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-contact'], 'success', $apiResponse); | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-tags'], $reponseType, $tagResponse); |
| @@ -86,7 +85,6 @@ jobs: | |||
| checks: | | |||
| i18n_usage | |||
| late_escaping | |||
There was a problem hiding this comment.
Pull request overview
This PR introduces a new Creator LMS integration across the frontend and backend, wires it into the existing integrations UI/flow system, and includes a few related UI/layout, integration helper, and CI build adjustments.
Changes:
- Added full Creator LMS integration (UI steps, field mapping, authorization, edit support) plus backend controller/routes and execution helper.
- Updated several integration UIs to use a larger step container
minHeight(200px → 500px). - Refactored parts of Constant Contact/Keap field-map initialization and adjusted CI build/plugin-check workflow.
Reviewed changes
Copilot reviewed 36 out of 37 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/resource/img/integ/creatorLms.webp | Adds Creator LMS logo asset used by dynamic logo loading. |
| frontend/src/components/Flow/New/SelectAction.jsx | Adds CreatorLms to selectable integration actions. |
| frontend/src/components/AllIntegrations/WPCourseware/WPCourseware.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/WPCafe/WPCafe.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/UserRegistrationMembership/UserRegistrationMembership.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/Telegram/Telegram.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/TeamsForWooCommerceMemberships/TeamsForWooCommerceMemberships.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/SeoPress/SeoPress.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/NewInteg.jsx | Lazy-loads and routes to new CreatorLms integration component. |
| frontend/src/components/AllIntegrations/MailerPress/MailerPress.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/MailPoet/MailPoet.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/Keap/KeapFieldMap.jsx | Removes inline field_map auto-regeneration logic; adjusts imports. |
| frontend/src/components/AllIntegrations/Keap/KeapCommonFunc.js | Ensures field_map is regenerated during custom field refresh. |
| frontend/src/components/AllIntegrations/IntegInfo.jsx | Adds Creator LMS authorization info component wiring. |
| frontend/src/components/AllIntegrations/FluentCart/FluentCart.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/FluentCRM/FluentCrm.jsx | Increases step container minHeight to 500px. |
| frontend/src/components/AllIntegrations/EditInteg.jsx | Adds edit routing/component for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/staticData.js | Defines Creator LMS actions/modules and field definitions for mapping UI. |
| frontend/src/components/AllIntegrations/CreatorLms/EditCreatorLms.jsx | Adds edit screen for Creator LMS integration configs. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsIntegLayout.jsx | Adds action selection + field mapping layout for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsFieldMap.jsx | Implements Creator LMS field mapping row UI. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsCommonFunc.js | Adds common helpers for input handling, mapped-field checks, and map generation. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLmsAuthorization.jsx | Adds authorization/activation check step for Creator LMS. |
| frontend/src/components/AllIntegrations/CreatorLms/CreatorLms.jsx | Adds the 3-step Creator LMS integration flow (auth → map → save). |
| frontend/src/components/AllIntegrations/ConstantContact/ConstantContactFieldMap.jsx | Removes inline field_map auto-regeneration logic; import cleanup. |
| frontend/src/components/AllIntegrations/ConstantContact/ConstantContactCommonFunc.js | Refactors generateMappedField signature to accept fields array. |
| frontend/src/components/AllIntegrations/ConstantContact/ConstantContact.jsx | Initializes ConstantContact field_map via generateMappedField(constantContactFields). |
| frontend/src/components/AllIntegrations/Autonami/Autonami.jsx | Increases step container minHeight to 500px. |
| frontend/src/Utils/StaticData/webhookIntegrations.js | Adds CreatorLms to customFormIntegrations list. |
| frontend/src/Utils/StaticData/tutorialLinks.js | Adds Creator LMS documentation link entry. |
| backend/Core/Util/AllTriggersName.php | Registers Creator LMS name/flags in the entity name list. |
| backend/Actions/Keap/RecordApiHelper.php | Adjusts logging and tag insertion flow for Keap contact creation. |
| backend/Actions/CreatorLms/Routes.php | Adds AJAX route creator_lms_authorize. |
| backend/Actions/CreatorLms/RecordApiHelper.php | Adds execution helper to dispatch Creator LMS actions via hooks + logs. |
| backend/Actions/CreatorLms/CreatorLmsController.php | Adds controller for auth check and integration execution. |
| .github/workflows/plugin-check.yml | Uses .github/build script and changes plugin-check checks list. |
| .github/build | Updates build packaging script (no longer copies build-hash.txt). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
145
to
+149
| if ($defaultConf->actions->tags || isset($apiResponse->id)) { | ||
| $tagResponse = $this->insertTag($apiResponse->id, $defaultConf->selectedTags); | ||
| } | ||
| $reponseType = HttpHelper::$responseCode === 200 ? 'success' : 'error'; | ||
|
|
||
| if (!(isset($apiResponse->id))) { | ||
| LogHandler::save($this->_integrationID, ['type' => 'contact', 'type_name' => 'add-contact'], 'error', $apiResponse); | ||
| } else { | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-contact'], 'success', $apiResponse); | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-tags'], $reponseType, $tagResponse); |
Comment on lines
+147
to
+149
| $reponseType = HttpHelper::$responseCode === 200 ? 'success' : 'error'; | ||
|
|
||
| if (!(isset($apiResponse->id))) { | ||
| LogHandler::save($this->_integrationID, ['type' => 'contact', 'type_name' => 'add-contact'], 'error', $apiResponse); | ||
| } else { | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-contact'], 'success', $apiResponse); | ||
| LogHandler::save($this->_integrationID, ['type' => 'record', 'type_name' => 'add-tags'], $reponseType, $tagResponse); |
| i18n_usage | ||
| late_escaping | ||
| plugin_header | ||
| plugin_readme |
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.
No description provided.