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

fix: php warning on PayPal response #1455

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sapayth
Copy link
Member

@sapayth sapayth commented Apr 30, 2024

After PayPal payment, when he click to return to the website user are facing critical error in PayPal payment.

@Rubaiyat-E-Mohammad
Copy link

Rubaiyat-E-Mohammad commented May 2, 2024

Issue-1:
Subscription pack and new transaction are not added

Scenario:

  1. From frontend, Log in as a user
  2. Try to add subscription pack
  3. Complete payment with PayPal
  4. Check subscription pack for the user
  5. Log out as a user
  6. Log in as an admin
  7. Go to admin dashboard
  8. Click on wp user frontend from side
  9. Click on subscription
  10. Observe that new subscription is not added
  11. Click on transaction
  12. Observe that no new transaction happened using Paypal

Screenshot for reference:

Screenshot 2024-05-02 at 5 34 13 PM Screenshot 2024-05-02 at 5 40 39 PM Screenshot 2024-05-02 at 5 43 18 PM

Debug-log:

[02-May-2024 11:02:01 UTC] PHP Notice:  Undefined index: subscription.phpyear in /Users/rubaiyatemohammad/QA-Manual-weDevs/SITES/wpuf_rub/wp-content/plugins/wp-user-frontend/includes/Admin/Admin_Subscription.php on line 719

@sapayth sapayth added the needs: developer feedback This PR needs developer feedback or code changes label May 3, 2024
@sapayth
Copy link
Member Author

sapayth commented May 6, 2024

The issue is related to our subscription pack creation process. we are saving some data in bad format. hopefully this will be resolved in our recent subscription page revamp PR

@sapayth sapayth removed the needs: developer feedback This PR needs developer feedback or code changes label May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants