Skip to content

add the posibility to the admin to get subscription for a specific user with a email#1031

Open
JamesDAdams wants to merge 3 commits intoellite:mainfrom
JamesDAdams:feat/get-subscription-from-email
Open

add the posibility to the admin to get subscription for a specific user with a email#1031
JamesDAdams wants to merge 3 commits intoellite:mainfrom
JamesDAdams:feat/get-subscription-from-email

Conversation

@JamesDAdams
Copy link
Copy Markdown
Contributor

@JamesDAdams JamesDAdams commented Mar 27, 2026

Hi, i made this. because i'm developping a Organizr plugin for show the subscription on Organizr dashboard for every users :

image

I've added an option image_base64 too for return the logo in a base64

Copilot AI review requested due to automatic review settings March 27, 2026 18:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an admin-only capability to retrieve subscriptions for a specific user by providing that user’s email to the existing subscriptions listing endpoint.

Changes:

  • Documented a new member_email request parameter for the subscriptions endpoint.
  • Added logic to resolve member_emailtargetUserId (admin-only).
  • Updated the subscriptions query to filter by targetUserId instead of the calling user.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread api/subscriptions/get_subscriptions.php
Comment thread api/subscriptions/get_subscriptions.php
Comment thread api/subscriptions/get_subscriptions.php
…d create docker-compose file for local dev
$response = [
"success" => true,
"title" => "subscriptions",
"title" => "subscriptions3",
Copy link
Copy Markdown
Owner

@ellite ellite Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this file.

Comment thread local-dev/.gitignore
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants