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: update script button overlaps with publish target content in setting page #5306

Merged
merged 3 commits into from
Dec 11, 2020

Conversation

liweitian
Copy link
Contributor

Description

image

Task Item

Closes #5302

Screenshots

settingUX

@coveralls
Copy link

coveralls commented Dec 11, 2020

Coverage Status

Coverage decreased (-0.002%) to 55.262% when pulling 49052eb on liweitian:fixSettingsUX into c8e6a75 on microsoft:main.

@boydc2014
Copy link
Contributor

Can you also fix the logic that, i don't think we want to show this ux when there is not "scripts" folder

@boydc2014
Copy link
Contributor

Can you also fix the logic that, i don't think we want to show this ux when there is not "scripts" folder

ignore this,i saw the other PR is doing that

@boydc2014 boydc2014 merged commit b9bb2de into microsoft:main Dec 11, 2020
alanlong9278 added a commit that referenced this pull request Dec 17, 2020
* main: (22 commits)
  fix: Added missing dependencies to /types and /extensions packages. (#4436)
  fix: Updated publish extensions to use updated APIs (#5346)
  docs: Supported/Recommended OS Documentation (#5330)
  chore: suppress/fix lgtm alerts (#4678)
  v1.3.0
  release: update changelog
  release: update changelog github user map
  RC5 (#5327)
  fix: security analysis failed because of ini@1.3.5 (#5326)
  fix: undo failed after click create trigger and remove trigger (#5325)
  fix: LG diagnostics mismatch by update reference in cache  (#5323)
  fix: Start panel open on start/restart operation (#5322)
  fix sentence wrap bug (#5310)
  fix: use dropdown for LUIS region in skill bot settings (#5318)
  chore: Updated to latest lib (#5316)
  feat: provisioning in app (#4221)
  surface dotnet build error on runtime (#5314)
  fix: runtime throw exception when there is no cosmosdb config (#5313)
  chore: Remove onnxruntime.pdb from Publishing (#5291)
  fix UX bug in setting page (#5306)
  ...
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
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 settings overlapping with publish settings
3 participants