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] [Modules] Avoid the implicit filters can be removables #3777

Merged

Conversation

Desvelao
Copy link
Member

@Desvelao Desvelao commented Jan 3, 2022

Description

This PR hides the button of filters to remove them from the search bar.

image

Changes

  • Use x.meta.value as function instead of use the implementation of toString when interpolating the value of x.meta.value

Tests

  • Go to Modules/Office 365/Panel or Modules/GitHub/Panel and access a drill-down view. A new filter will be added with the accessed value, the button to remove the filters should not be visible

Closes #3769

  - Use `x.meta.value` as function instead of use the implementation of `toString` when interpolating the value of `x.meta.value`
@Desvelao Desvelao added the type/bug Bug issue label Jan 3, 2022
@Desvelao Desvelao requested a review from a team January 3, 2022 15:00
@Desvelao Desvelao self-assigned this Jan 3, 2022
@Desvelao Desvelao changed the title [FIX] Fix/3769 avaid remove implicit filter drilldown panel [FIX] [Modules] Avoid the implicit filters can be removables Jan 3, 2022
@Desvelao Desvelao added the 4.3 label Jan 10, 2022
@Desvelao Desvelao mentioned this pull request Jan 10, 2022
7 tasks
Copy link
Member

@asteriscos asteriscos left a comment

Choose a reason for hiding this comment

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

Test: ✔️
CR: ✔️
LGTM!

image

@yenienserrano yenienserrano self-requested a review January 11, 2022 19:33
Copy link
Member

@yenienserrano yenienserrano left a comment

Choose a reason for hiding this comment

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

CR: ✔️
Test: ✔️
LGTM

@Desvelao Desvelao merged commit 5cef1c6 into 4.3-7.16 Jan 13, 2022
@Desvelao Desvelao deleted the fix/3769-avaid-remove-implicit-filter-drilldown-panel branch January 13, 2022 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Bug issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

allows deleting the user filter in the module dashboard of the office 365
3 participants