Automattic / Automattic/woocommerce-services
The plugin should be Jetpack Developer Mode aware
- 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
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