Automattic / Automattic/jetpack

Factor generic logic out of security/index.jsx

Open
#14,114 2 comments 0 reactions 0 assignees View on GitHub
[Feature] Scan [Plugin] Jetpack [Type] Janitorial Admin Page
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

These pieces of logic will likely be useful for helpers in other places of the code, so it would be useful to extract them as separate helper functions, but that's totally non-mandatory for now, and can be done in another PR.

_Originally posted by @tyxla in https://github.com/Automattic/jetpack/pull/14098/files_

Contributor guide

Open the contributing guide

Research direction

Read security/index.jsx and the discussion or diff from pull request 14098 to identify which logic is generic and which parts are specific to the security screen. Done means the reusable logic is separated without changing current behavior; the issue does not name a test file or verification command.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
frontend, security
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.