Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@szakarias
Copy link
Contributor

No description provided.

Copy link
Contributor

@jakobr-google jakobr-google left a comment

Choose a reason for hiding this comment

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

LGTM

[((UINavigationController*)viewController) popViewControllerAnimated:NO];
}

}
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: This one probably shouldn't be indented.

"framework/Headers/FlutterMacros.h",
"framework/Headers/FlutterPlugin.h",
"framework/Headers/FlutterViewController.h",
"framework/Headers/FlutterNavigationController.h",
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe you'll need to add the new files to some licenses index file as well. Travis would know for sure.

@Hixie
Copy link
Contributor

Hixie commented Jul 24, 2017

Can we test this?

@Hixie
Copy link
Contributor

Hixie commented Aug 8, 2017

@szakarias is this still being worked on? If not, can we close it in the meantime, so that it is removed from the review queue?

@szakarias
Copy link
Contributor Author

I'll merge this tomorrow CEST. I'm working on a test for it in a follow-up PR.

@szakarias szakarias merged commit 2b7a8f1 into flutter:master Aug 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants