Skip to content

Conversation

@rafaelblink
Copy link
Contributor

@rafaelblink rafaelblink commented Mar 25, 2021

Proposed changes (including videos or screenshots)

After migration to the new edit room component, some fields are wrong, this PR will show correct fields such as custom fields (room scope), priority (EE), etc.

Before

image

After

image

Issue(s)

Steps to test or reproduce

Further comments

@rafaelblink rafaelblink marked this pull request as ready for review March 26, 2021 13:15
@renatobecker renatobecker changed the title Regression: Fix inputs in Edit Room Form Regression: Fix inputs in Omnichannel Edit Room Form Mar 26, 2021
@renatobecker renatobecker changed the title Regression: Fix inputs in Omnichannel Edit Room Form Regression: New Room editing form not rendering priority and custom fields Mar 30, 2021
@renatobecker renatobecker changed the title Regression: New Room editing form not rendering priority and custom fields Regression: New Room Edit form not rendering priority and custom fields Mar 30, 2021
Copy link
Contributor

@renatobecker renatobecker left a comment

Choose a reason for hiding this comment

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

When I open the Room Edit form and I change custom fields, the Confirm button doesn't get enabled.

Screen Shot 2021-03-30 at 1 38 06 PM

Also, there are other cases when I open the same for and the Confirm button is enabled by default.

We need to fix all these inconsistencies.

@renatobecker renatobecker changed the title Regression: New Room Edit form not rendering priority and custom fields Regression: Room Edit form not rendering priority and custom fields Mar 31, 2021
@renatobecker
Copy link
Contributor

Screen Shot 2021-03-31 at 12 17 15 PM

Also, when I run the server without an enterprise license I got the error displayed above.

@renatobecker renatobecker merged commit 8ba4c4a into develop Mar 31, 2021
@renatobecker renatobecker deleted the omnichannel/inputs-edit-room-regression branch March 31, 2021 19:12
@sampaiodiego sampaiodiego mentioned this pull request Apr 3, 2021
13 tasks
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.

3 participants