Skip to content

[NCPS Hosted buttons] Update FPTI tracking params #2557

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Sep 30, 2025
Merged

[NCPS Hosted buttons] Update FPTI tracking params #2557

merged 4 commits into from
Sep 30, 2025
+20 −59

Conversation

masuthar
Copy link
Contributor

@masuthar masuthar commented Sep 16, 2025 •
edited
Loading

What is the purpose of this PR?

Jira Ticket:

[DTNCPS-2836]

Type of change

  • New feature (backward compatible change that adds new capability).
  • UI change
  • Bug fix.
  • Breaking change (backward incompatible change). Provide references to customer impact and communication.
  • Refactor (no functional change)

Testing Plan

Below is information that validates the successful implementation of this feature and how a release manager can validate the success of a release candidate in the absence of the PR author.

PR Author: https://paypal.enterprise.slack.com/team/U08242F8JUE

Backup Validator: https://paypal.enterprise.slack.com/team/WCUPVFGD7

PR Author's Team: NCNW-Cosmos, https://paypal.enterprise.slack.com/team/W3Y296T0D

Step-by-Step Validation

  1. Step one: Create a NCPS Button (https://www.te-develop-ncnw.qa.paypal.com/ncp/links/create)
    Screenshot 2025-07-08 at 2 42 41 PM

-> build it

Screenshot 2025-07-08 at 2 42 53 PM

  1. Step two: Copy Paste the code in a html file and update sdk url to be of Test Env in the copy code section and open in browser (using any web server) and proceed to pay using Paypal checkout

Example snippet:

<html>
  <script src="https://www.te-ncpsjssdk.qa.paypal.com/sdk/js?client-id=B_AGokdO0X6Oc3uJ8NeL16UEjncvaCWjWqVShwzNrnBdHcu99GjAQfaOdTs2F87t9dHg8jJ9XrZIYIUPp8&components=hosted-buttons&enable-funding=venmo&currency=USD"></script>
  <div id="paypal-container-7V79Y46L2UNZG"></div>
  <script>
    paypal
      .HostedButtons({
        hostedButtonId: "7V79Y46L2UNZG",
      })
      .render("#paypal-container-7V79Y46L2UNZG");
  </script>
</html>

Before/After Screenshots

Screen.Recording.2025-09-16.at.1.34.41.PM.mov

DataDog Graph to Monitor

CAL Logs & Regex

Rollback Considerations

Are there any services dependent on this change?

  • Yes
  • No

@masuthar masuthar requested a review from a team as a code owner September 16, 2025 02:44
@masuthar masuthar merged commit 38b1bd4 into main Sep 30, 2025
4 checks passed
@masuthar masuthar deleted the feature/ncpsfpti branch September 30, 2025 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ravishekhar ravishekhar ravishekhar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@masuthar @ravishekhar