-
Notifications
You must be signed in to change notification settings - Fork 36
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(studio): allow changing content-type when editing node content-element #121
Conversation
DEV-1535 "[BUG]: Failing to change Content Element" (botpress/botpress botpress/v12#1430)
Describe the bug To Reproduce
Expected behavior Environment (please complete the following information):
botpress/botpress botpress/v12#1430 by @Michael-N-M on GitHub via LinearSync |
I see pretty much only typing fix, cab you point out where the actual fix is in your changes |
renderCurrentCategoryInfo() { | ||
const { categories } = this.props | ||
const { hideCategoryInfo } = this.state | ||
if (hideCategoryInfo || !categories || categories.length < 2) { | ||
if (!categories || categories.length < 2) { | ||
return null | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is the fix. I simply removed the hideCategoryInfo
condition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I thought that. Any idea why was it there in the first place ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no idea!
This PR fixes an issue where when editing a selected content-element in a node, it would not let you change the content-type. The option was only provided when adding a content-element.
Before
simplescreenrecorder-2021-09-24_10.58.25.mp4
After
simplescreenrecorder-2021-09-24_10.56.37.mp4
Closes botpress/v12#1430
Also closes DEV-1535