-
Notifications
You must be signed in to change notification settings - Fork 1
Feat/fabman-action #116
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
base: main
Are you sure you want to change the base?
Feat/fabman-action #116
Conversation
This reverts commit e4b0bfc.
frontend-dev/src/components/AllIntegrations/Fabman/EditFabman.jsx
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/EditFabman.jsx
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/EditFabman.jsx
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/IntegrationHelpers.jsx
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/IntegrationHelpers.jsx
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/IntegrationHelpers.jsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a new Fabman integration to the system, enabling users to manage members and spaces through the Fabman API.
- Introduces comprehensive Fabman API integration with authentication, workspace management, and CRUD operations for members and spaces
- Adds complete frontend UI components for configuration, field mapping, and workflow management
- Implements backend API routes and controllers for Fabman operations with proper error handling
Reviewed Changes
Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| includes/Actions/Fabman/Routes.php | Defines API routes for Fabman authorization and workspace fetching |
| includes/Actions/Fabman/RecordApiHelper.php | Core API helper class handling all Fabman API operations and data mapping |
| includes/Actions/Fabman/FabmanController.php | Main controller managing authentication, workspace fetching, and integration execution |
| frontend-dev/src/components/Flow/New/SelectAction.jsx | Adds Fabman to the list of available integrations |
| frontend-dev/src/components/AllIntegrations/NewInteg.jsx | Registers Fabman component for new integration flow |
| frontend-dev/src/components/AllIntegrations/IntegInfo.jsx | Adds Fabman authorization component to integration info |
| frontend-dev/src/components/AllIntegrations/Fabman/* | Complete frontend implementation including authorization, field mapping, and configuration components |
| frontend-dev/src/components/AllIntegrations/EditInteg.jsx | Adds Fabman to the edit integration flow |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
frontend-dev/src/components/AllIntegrations/Fabman/FabmanCommonFunc.js
Outdated
Show resolved
Hide resolved
frontend-dev/src/components/AllIntegrations/Fabman/EditFabman.jsx
Outdated
Show resolved
Hide resolved
| import { fabmanAuthentication } from './FabmanCommonFunc' | ||
| import Note from '../../Utilities/Note' | ||
| import tutorialLinks from '../../../Utils/StaticData/tutorialLinks' | ||
| import TutorialLink from '../../Utilities/TutorialLink' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add Fabeman documentation link to TutorialLink file
No description provided.