Skip to content

Ss/collapsed issue boards per user ce

Scott Stern requested to merge ss/collapsed-issue-boards-per-user-ce into master

What does this MR do?

Fixes frontend req for: https://gitlab.com/gitlab-org/gitlab-ce/issues/52758

We needed to make a request including the collapsed state if the user was logged in, if theyre not logged in we default to localStorage.

Testing thoughts as I am going through this MR:

  • In board_spec.js we should make a issue to move the list specific tests in this file into the list_spec file.
  • We should make an issue to move the board_spec test file to jest (i dont mind taking this on).
  • I apologize for all of the changes in board_spec, we needed to reformat the test file for simplicity when porting over to jest, separating concerns, and isolating when we mount the whole vue component or not.

Implementation thoughts:

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

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
Edited by 🤖 GitLab Bot 🤖

Merge request reports