Skip to content

Update readme and composer.json #1

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

Merged
merged 1 commit into from
Mar 6, 2014
Merged

Conversation

amacneil
Copy link
Contributor

I just went through and made the package name consistent for you. Before I list it in the main Omnipay repo, can you merge this pull request, and also rename the github repository to lokielse/omnipay-alipay (to make it consistent with the other omnipay gateways).

Then you will need to resubmit it to packagist.org with the new repository URL (just delete the current one).

lokielse added a commit that referenced this pull request Mar 6, 2014
Update readme and composer.json
@lokielse lokielse merged commit 566fe9b into lokielse:master Mar 6, 2014
@lokielse
Copy link
Owner

lokielse commented Mar 6, 2014

sorry for late, merged.

@asaph
Copy link
Contributor

asaph commented Mar 15, 2014

@adrianmacneil @lokielse Can this package be added to the main Omnipay repo yet?

@amacneil
Copy link
Contributor Author

Sure. Can you fix the travis link on the README? I think it's because this repository is named alipay-omnipay but the package is named omnipay-alipay. It would be good to be consistent, so probably rename the github package to omnipay-alipay so it matches the package (and other omnipay packages)

@lokielse
Copy link
Owner

thanks @adrianmacneil, I renamed github package slug just now.

@asaph
Copy link
Contributor

asaph commented Mar 24, 2014

@adrianmacneil @lokielse Just a gentle bump of this thread. Anything else we need to do to get this lib added to Omnipay?

@amacneil
Copy link
Contributor Author

Thanks for the bump, I've added to the main Omnipay readme now :)

@asaph
Copy link
Contributor

asaph commented Mar 27, 2014

@adrianmacneil Thanks!

@lokielse
Copy link
Owner

Thank both of you @adrianmacneil @asaph. There are some bugs may not find now. I'll try to fix them and intent to using it in production after a few days.

lokielse pushed a commit that referenced this pull request Dec 7, 2016
lokielse pushed a commit that referenced this pull request May 24, 2018
lokielse added a commit that referenced this pull request Mar 7, 2019
Update readme and composer.json
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse added a commit that referenced this pull request Mar 7, 2019
Update readme and composer.json
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse added a commit that referenced this pull request Mar 7, 2019
Update readme and composer.json
lokielse pushed a commit that referenced this pull request Mar 7, 2019
lokielse pushed a commit that referenced this pull request Mar 7, 2019
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