Migrate Form Plugin#357
Conversation
… and updated editor inheritance
# Conflicts: # src/plugins/index.js
benthongtiang
added
bug
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
johnny-hausman
left a comment
johnny-hausman
left a comment
There was a problem hiding this comment.
OK, I realize we'll have future issues with the hard coded links to existing files, but for now we can let those go.
|
|
||
| export default function FNAbviewformEditor({ AB, ABViewEditorPlugin }) { | ||
| const FABViewContainer = | ||
| require("../../rootPages/Designer/editors/views/ABViewContainer").default; |
There was a problem hiding this comment.
ok, in the #jh/mcp_server branch, we should pass in this value as it is a hard reference to an existing file.
But this isn't going into that branch, so leave it as is and make a comment to remind me to fix this.
:)
There was a problem hiding this comment.
Done! Added a comment to remind you later. Thanks for the review.
|
looks like there is probably a recursive loop in our loading of our components. 
|
… component management
The E2E test in GitHub is failing because it's running against the older platform code. Once we merge the matching ben/FormPlugin branch in ab_platform_web, the tests will pass. |
…iewForm dependencies
3 participants
Created the new web_view_form plugin structure inside the ABDesigner.
Fix designer display issue