-
Notifications
You must be signed in to change notification settings - Fork 62
Exporter UI Refactor [AARD-1883]
#1194
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: dev
Are you sure you want to change the base?
Conversation
553417a
to
cea6ff1
Compare
Chorus detected one or more security issues with this pull request. See the Checks tab for more details. As a reminder, please follow the secure code review process as part of the Secure Coding Non-Negotiable requirement. |
Also note that to test this, you'll need to build the react project. Not sure how we want to do that long-term because previously, using the exporter from the synthesis repo just worked out of the box cd exporter/SynthesisFusionAddin/web
bun i
bun run build |
Task
AARD-1883
Uses Fusion's palette system to make the UI for the exporter a React webpage. Goal is to make the UI cleaner, easier to build on, and not such a pain to update.
Symptom
When you click the exporter button in fusion, you previously saw the old UI. The old UI was difficult to update and very inflexible in its layout.
Solution
ConfigureCommand
GUI replaced with a palette, which has all of the same inputs re-implemented using MUI and ReactPhotos
Limitations
Verification
ExporterOptions
objects were the same with both the old configuration UI and the new web UIBefore merging, ensure the following criteria are met: