Skip to content

Packaging fixes and support for more fields.#1

Open
davel wants to merge 22 commits into
hembreed:masterfrom
davel:master
Open

Packaging fixes and support for more fields.#1
davel wants to merge 22 commits into
hembreed:masterfrom
davel:master

Conversation

@davel

@davel davel commented Nov 11, 2011

Copy link
Copy Markdown

Hi,

In this pull request there is various fixes and enhancements made by Tomas Doran, Ron Phipps and myself.

Kind regards,
Dave Lambley

@hembreed

Copy link
Copy Markdown
Owner

Thanks for this, I'll get it in along with some other fixes once they're tested. What version of Perl did you use?

On Nov 11, 2011, at 9:26 AM, davel wrote:

Hi,

In this pull request there is various fixes and enhancements made by Tomas Doran, Ron Phipps and myself.

Kind regards,
Dave Lambley

You can merge this Pull Request by running:

git pull https://github.com/davel/Business-PayPal-API master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • We are not in CVS, this is unused anywhere
  • META.yml should be a generated file, and therefore should not be kept in revision control
  • Add a .gitignore
  • Remove MANIFEST, this should also be a generated file
  • Add a MANIFEST.SKIP
  • Add repository metadata to the generated META.yml so that people know where to find the git repository
  • Add PayerName fields documented by PayPal.
  • "Suffix" a little too vague.
  • The payer's country is not necessarily the same as the shipping address. If
  • * Added support for protection eligbility
  • * Reduced required version of SOAP
  • Merge branch 'master' of git://github.com/bobtfish/Business-PayPal-API
  • Bump to mention currently release version.
  • Update changelog.
  • Add debian packaging.
  • Fix name.
  • Fix more names.

-- File Changes --

A .gitignore (9)
M Changes (10)
D MANIFEST (30)
A MANIFEST.SKIP (11)
D META.yml (11)
M Makefile.PL (5)
A debian/compat (1)
A debian/control (31)
A debian/copyright (14)
A debian/rules (81)
M lib/Business/PayPal/API.pm (3)
M lib/Business/PayPal/API/ExpressCheckout.pm (14)
M lib/Business/PayPal/API/GetTransactionDetails.pm (1)

-- Patch Links --

https://github.com/hembreed/Business-PayPal-API/pull/1.patch
https://github.com/hembreed/Business-PayPal-API/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

@davel

davel commented Dec 15, 2011

Copy link
Copy Markdown
Author

I'm currently testing with Perl 5.10.1.

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.

5 participants