Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Tracker URL field to WordCamp details report #958

Draft
wants to merge 1 commit into
base: production
Choose a base branch
from

Conversation

timiwahalahti
Copy link
Collaborator

This is an attempt to add new "Tracker URL" field to WordCamp details report. Couldn't test, since csv export utility is in mu-plugins-private for some reason.

Warning: require_once(/usr/src/public_html/wp-content/mu-plugins-private/wporg-mu-plugins/pub-sync/utilities/class-export-csv.php): failed to open stream: No such file or directory in /usr/src/public_html/wp-content/mu-plugins/2-autoloader.php on line 27

Either that utility needs to be moved to a public repository, or this PR can act as a base for someone who has access to private plugins. I guess the latter one will be easier and faster.

Fixes #823

Props @peiraisotta

How to test the changes in this Pull Request:

  1. Navigate to WordCamp Details report
  2. Do csv export, the field is selected by default

@pkevan
Copy link
Contributor

pkevan commented Feb 8, 2024

Either that utility needs to be moved to a public repository, or this PR can act as a base for someone who has access to private plugins. I guess the latter one will be easier and faster.

Turns out that the file in question is actually here: https://github.com/WordPress/wporg-mu-plugins/blob/trunk/mu-plugins/utilities/class-export-csv.php, but is built in a rather convoluted way (of which I don't know the history of). Is that helpful @timiwahalahti?

@renintw renintw added [Priority] High [Type] Good First Issue Straightforward, self-contained, doesn't require deep knowledge of codebase [Type] Enhancement and removed [Priority] 1 labels Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Reports [Priority] High [Type] Enhancement [Type] Good First Issue Straightforward, self-contained, doesn't require deep knowledge of codebase
Projects
Status: 📝 Draft
Development

Successfully merging this pull request may close these issues.

Add field "Tracker URL" to report available fields
3 participants