Automattic / Automattic/jetpack
Extract notice functions into a shared helper
Open
[Focus] Blocks
[Pri] Low
[Size] S
Enhancement
Task
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
As suggested by @jeherve in https://github.com/Automattic/jetpack/pull/14884#discussion_r388377243, we should add a shared helper function for setting error notices, to reduce duplicate code.
Contributor guide
Research direction
Start with the discussion in PR #14884 that proposed this issue, then locate the duplicated error-notice functions in the Jetpack codebase. Extract their shared behavior into one helper and confirm the existing notice callers use it without changing the resulting notices.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, wordpress
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100