Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

fix(board): Don't display closed issues in columns #666

Merged
merged 1 commit into from
Mar 25, 2021

Conversation

zemzale
Copy link
Collaborator

@zemzale zemzale commented Mar 25, 2021

Description
In web UI issues that are closed are moved into their own column. They
shouldn't belong in the main columns, if they are closed.

There is the thing that issues that don't belong in any of user
made boards, should be in their own boards.

Issue for adding the extra columns #665

Related Issue
Resolves #664

How Has This Been Tested?
Tested localy with board that showed closed issues, and now doesn't.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)

In web UI issues that are closed are moved into their own column. They
shouldn't belong in the main columns, if they are closed.

There is the thing that issues that don't belong in any of user
made boards, should be in their own boards.

Issue for adding the extra columns profclems#665

Fixed profclems#664
@codecov
Copy link

codecov bot commented Mar 25, 2021

Codecov Report

Merging #666 (aba2212) into trunk (6e21a96) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            trunk     #666   +/-   ##
=======================================
  Coverage   60.03%   60.03%           
=======================================
  Files          90       90           
  Lines        6438     6438           
=======================================
  Hits         3865     3865           
  Misses       2209     2209           
  Partials      364      364           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e21a96...aba2212. Read the comment docs.

Copy link
Owner

@profclems profclems left a comment

Choose a reason for hiding this comment

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

LGTM

@profclems profclems merged commit 4e78089 into profclems:trunk Mar 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants