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

fix: Hide deprecated "custom runtime" related features for adaptive rutime #6367

Merged
merged 17 commits into from
Mar 31, 2021

Conversation

benbrown
Copy link
Contributor

@benbrown benbrown commented Mar 10, 2021

Description

Since new Adaptive Runtime comes pre-ejected, we no longer support the ability to eject, nor do we support the ability to toggle this feature off. All that remains in the runtime control section is the customizable start command that is used when the bot is started. It is very useful for this to remain available as it allows the developer to pass additional options along to the runtime.

  • Removes "Use custom runtime" option from settings for new-style Adaptive runtime bots
  • Removes ability to eject for new-style Adaptive runtime bots
  • Removes ability to change the path to the runtime

Task Item

Fixes #6321

Screenshots

For "old" bots, no change:

Screen Shot 2021-03-10 at 5 04 56 PM

For "new" bots, removed features:

Screen Shot 2021-03-10 at 5 11 12 PM

@coveralls
Copy link

coveralls commented Mar 18, 2021

Coverage Status

Coverage decreased (-0.01%) to 52.059% when pulling 114e5d8 on benbrown/runtimeopts into 9a3dd98 on main.

Copy link
Member

@GeoffCoxMSFT GeoffCoxMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to update the isAdaptive dependencies (or useMemo) and consider calling the shared method of isUsingAdaptiveRuntime.

@benbrown benbrown requested a review from GeoffCoxMSFT March 26, 2021 23:17
@benbrown benbrown merged commit 586d33d into main Mar 31, 2021
@benbrown benbrown deleted the benbrown/runtimeopts branch March 31, 2021 21:50
@cwhitten cwhitten mentioned this pull request May 20, 2021
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
…utime (microsoft#6367)

* Hide deprecated "custom runtime" related features for adaptive rutime

* hide ability to change path

* fix type

* remove lib files

* updates

* use isusingadpativeruntime

* fix usememo syntax
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.

Custom Runtime section still in settings menu
3 participants