Automattic / Automattic/woocommerce-services

The plugin should be Jetpack Developer Mode aware

Open
#847 1 comment 0 reactions 0 assignees View on GitHub
[Type] Technical Debt woocommerce-services
Dominant language
JavaScript
Stars
112
Forks
31
Avg merge
3d 1h
Merged PRs (30d)
14

Description

We are currently displaying the "account settings" page for our plugin if we detect a Jetpack in Developer Mode.. why?

See: https://github.com/Automattic/woocommerce-services/blob/master/classes/class-wc-connect-settings-pages.php#L133

This seems like code only to support testing - and while it's convenient, it's wholly unnecessary.

Contributor guide

Open the contributing guide

Research direction

Open classes/class-wc-connect-settings-pages.php around line 133 and trace the Jetpack Developer Mode check that displays the account settings page. Confirm the behavior is only testing support, then remove or adjust it so Developer Mode no longer triggers that page; verify the plugin's settings behavior afterward.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.