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

Add order default notify_url. #325

Merged
merged 1 commit into from
Mar 6, 2016
Merged

Add order default notify_url. #325

merged 1 commit into from
Mar 6, 2016

Conversation

foreverglory
Copy link
Contributor

$options = [
    // ...
    'payment' => [
        // ...
        'notify_url' => 'http://xxx.com/order-notify';
    ]
];
$app = new Application($options);
$attributes = [
    // ...
    'notify_url'       => 'http://xxx.com/order-notify';,

];
$order = new Order($attributes);

if $order haven't notify_url attribute, use $options['payment']['notify_url']. Don't need to be set notify_url in every order.

$options = [
    // ...
    'payment' => [
        // ...
        'notify_url' => 'http://xxx.com/order-notify';
    ]
];
$app = new Application($options);
$attributes = [
    // ...
    'notify_url'       => 'http://xxx.com/order-notify';,

];
$order = new Order($attributes);

if $order haven't notify_url attribute, use $options['payment']['notify_url']. Don't need to be set notify_url in every order.
overtrue added a commit that referenced this pull request Mar 6, 2016
Add payment default notify_url.
@overtrue overtrue merged commit 2012691 into w7corp:develop Mar 6, 2016
@overtrue overtrue changed the title add payment default notify_url. Add payment default notify_url. Mar 6, 2016
@overtrue overtrue changed the title Add payment default notify_url. Add order default notify_url. Mar 6, 2016
overtrue added a commit that referenced this pull request Mar 6, 2016
@overtrue
Copy link
Collaborator

overtrue commented Mar 6, 2016

@foreverglory 在 issue 或者 pull request 的 评论里,使用代码块的方式:

```php
这里放 PHP 代码
```

这样代码就会像我给你改过的样子一样高亮了。:smile:

overtrue added a commit to EasyWeChat/docs that referenced this pull request Mar 7, 2016
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.

2 participants