Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Ensure onCreate calls through to super onCreate#53

Closed
georgepapas wants to merge 1 commit intofacebook:masterfrom
georgepapas:quick_start_activity
Closed

Ensure onCreate calls through to super onCreate#53
georgepapas wants to merge 1 commit intofacebook:masterfrom
georgepapas:quick_start_activity

Conversation

@georgepapas
Copy link
Contributor

Quick start doco creates an activity which does not call through to super.onCreate and causes the sample Hello World app to crash.

@LegNeato
Copy link
Contributor

Can you rebase and sign the Facebook Contributors License Agreement @ http://developers.facebook.com/opensource/cla ? You don't need a Facebook account to sign. FYI, we adopted the Apache Software Foundation's CLA which is also used by Google, RedHat, Zend, and others.

@georgepapas
Copy link
Contributor Author

Hi,

Signed agreement, branch is up to date.

Thanks

@LegNeato
Copy link
Contributor

Awesome! We'll work on merging this asap.

bolinfest pushed a commit that referenced this pull request Nov 20, 2013
Summary:
Quick Start docs create an activity that does not call through to
super.onCreate and causes the sample Hello World app to crash.

#53

Test Plan: http://localhost:9811/setup/manual_quick_start.html
@bolinfest
Copy link
Contributor

Patched as 10f50e2.

@bolinfest bolinfest closed this Nov 20, 2013
@facebook-github-bot
Copy link
Contributor

@georgepapas updated the pull request.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants