mattermost-community/focalboard

Webapp Linter Fixes - Investigate/Cleanup ToDos

Open

#1,346 opened on Sep 28, 2021

View on GitHub
 (2 comments) (0 reactions) (1 assignee)TypeScript (2,542 forks)user submission
HacktoberfestHelp Wanted

Repository metrics

Stars
 (26,187 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

There are several files with ToDos left in the code. These need to be investigated, fixed and removed. * Run npx eslint --ext .tsx,.ts . in the webapp folder to see them

If small enough and easy enough to understand and fix. They can be fixed as part of this ticket. However, if after investigating the issue is large and/or complicated, a separate ticket should be created and this ticket can be closed by referencing the new ticket and removing the TODO from code.

This ticket includes the investigation of the following files. /webapp/src/archiver.ts /webapp/src/cardFilter.ts

Contributor guide

Research direction
Run `npx eslint ext .tsx,.ts .` in the webapp folder to identify all TODO comments in the specified files. For each TODO, evaluate if it can be fixed easily. If so, implement the fix and remove the TODO. If not, create a new issue referencing the TODO and remove it from the code.
Tech stack
typescript
Domain
frontend
Issue type
Chore
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitNode.jsnpm
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70