Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Stripe Stripe
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 75
    • Issues 75
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Extensions
  • StripeStripe
  • Merge requests
  • !27

Mr23+

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed phani requested to merge phani/stripe:MR23+ into master Apr 03, 2019
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 8

second try: feel free to tell me i'm crazy and close this MR, but for me the code works now -- even though i still don't really know what i'm doing i'm afraid.

this includes !23 (merged) and is supposed to be merged into master. i only changed Customer.php; the other changes come from !23 (merged) .

i removed is_live from $RequiredParams from all applicable functions, and now new contacts are created in stripe and CiviCRM, and old ones are updated if applicable.

i'm still having problems with one particular contact that exists in civic but not in stripe, but that's probably due to this being a live civic instance and i can't go and remove things as i want.

Edited Apr 03, 2019 by phani
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: MR23+