Refactor(VSA filter bar): use general filter bar
What does this MR do?
This MR changes the VSA (cycle analytics) page to use the general filtered_search_bar.vue instead of our own implementation.
This is a stepping stone towards implementing a general filter bar solution for analytics.
In changing to use this general implementation, we get recent searches feature for free. No ticket was created for this effort. It also fixes an issue where the filter would not get populated properly from the URL (this is a recent bug), see screenshots. In order to do this, a few actions needed to be corrected.
In order to use this general solution, the filter bar is moved to its own row to give it more space for the many options.
Note: This filter bar is behind the following feature flags: value_stream_analytics_path_navigation
and value_stream_analytics_filter_bar
. Ensure they are turned on before testing this out. This is also the reason why there is no changelog entry.
Screenshots
Before: | After: |
---|---|
Does this MR meet the acceptance criteria?
Conformity
- [-] Changelog entry Not needed as this is still behind a feature flag, which is being rolled out in a different MR
- [-] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines -
Style guides - [-] Database guides
-
Separation of EE specific content
Availability and Testing
This implementation is fairly safe as it uses a component that is in use in other places, over a specific implementation of the same. The feature tests also pass with very few modifications, which is reassuring.
To test this out navigate to, e.g. this URL: http://127.0.0.1:3000/groups/flightjs/-/analytics/value_stream_analytics?project_ids%5B%5D=6&created_after=2020-07-19&created_before=2020-08-17&milestone_title=Upcoming&label_name%5B%5D=Bryffe&assignee_username%5B%5D=root
Known issues:
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers - [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
- [-] Label as security and @ mention
@gitlab-com/gl-security/appsec
- [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
- [-] Security reports checked/validated by a reviewer from the AppSec team
Related to #232465 (closed)