Skip to content
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

feat: Added configuration editor step #8090

Merged
merged 6 commits into from Jun 17, 2021

Conversation

VamsiModem
Copy link
Contributor

@VamsiModem VamsiModem commented Jun 15, 2021

Description

  1. Added configuration editor step
    - Removed @bfc/code-editor dependency in favor of Monaco.
  2. Made recoil atoms more granular.
  3. Moved extension initialization to a hook.
  4. Added JSON Editor

Task Item

refs #5744
refs #7632

Screenshots

image
image

@coveralls
Copy link

coveralls commented Jun 15, 2021

Coverage Status

Coverage increased (+1.2%) to 51.202% when pulling e070a99 on vmodem/configuration-editor-step into f2f7684 on main.

@VamsiModem VamsiModem merged commit 4cdbfd3 into main Jun 17, 2021
@VamsiModem VamsiModem deleted the vmodem/configuration-editor-step branch June 17, 2021 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants