Part 2 - Add "use_legacy_web_ide" to "user_preference"
- Draft: Add user setting to toggle between legac... (!98295 - closed)
- ...
- Part 1 - Hide top nav with vscode_web_ide featu... (!98891 - merged)
- Part 2 - Add "use_legacy_web_ide" to "user_pref... (!98945 - merged)
- Part 3 - Add "Switch to new Web IDE" in legacy ... (!102146 - merged)
What does this MR do and why?
In this MR, we support #370139 (closed) by:
- Introducing a
user_preference
foruse_legacy_web_ide
- Updating the
ide_data
to pass ause_vscode_web_ide
only if this new flag is false☝ and the feature is enabled
Screenshots or screen recordings
With Feature.enable(:vscode_web_ide)
User's use_legacy_web_ide=true
|
User's use_legacy_web_ide=false
|
---|---|
How to set up and validate locally
Confirm that, the VSCode Web IDE loads by default when Feature.enable(:vscode_web_ide)
.
- In a rails console (
rails c
), runFeature.enable(:vscode_web_ide)
. - Sign into
root
and visit Web IDE for a non-empty project. - expect: The new VSCode Web IDE should be loaded.
Confirm that, the legacy Web IDE will load if Feature.enable(:vscode_web_ide)
but the user's use_legacy_web_ide
is true.
- In the rails console (
rails c
), runUser.find(1).update!(use_legacy_web_ide: true)
. - Sign into
root
and visit Web IDE for a non-empty project. - expect: The legacy VSCode Web IDE should be loaded.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #370139 (closed)
Edited by Paul Slaughter