Skip to content

Feature/push callback#1297

Closed
CheungSKei wants to merge 11 commits intoalibaba:masterfrom
CheungSKei:feature/push_callback
Closed

Feature/push callback#1297
CheungSKei wants to merge 11 commits intoalibaba:masterfrom
CheungSKei:feature/push_callback

Conversation

@CheungSKei
Copy link
Copy Markdown
Contributor

新增分支

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 23, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ shaoji.zsj
❌ CheungSKei


shaoji.zsj seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@CheungSKei CheungSKei closed this Jul 23, 2021
@CheungSKei CheungSKei reopened this Feb 14, 2022
/// If you want to complete the push finish event with some custom data,
/// you can resolve a [result] object with [handler.resolve].
///
void onPushFinish(BoostInterceptorOption option, PushInterceptorHandler handler) =>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

与onPrePush呼应,用onPostPush可能会更合适点?

refreshOnPush(container);
}

_pushFinish(pageName, arguments: arguments, uniqueId: uniqueId, withContainer: true);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

内部路由(withContainer=false)走不到?

@0xZOne 0xZOne mentioned this pull request Feb 15, 2022
@0xZOne
Copy link
Copy Markdown
Collaborator

0xZOne commented Feb 16, 2022

已合并到 #1583

@0xZOne 0xZOne closed this Feb 16, 2022
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