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

HowItWorks and OnActivityResult Document Improvement #1580

Closed

Description

@Francis0121

Hi~~
When I used startForResult method, I didn't know that exist OnActivityResult Annotation. So I implemented all override method.

I think that HowItWorks page will give this annotation info, many people will use this.

So I change documentation HowItWorks page like this link.

https://github.com/JuranoSaurus/androidannotations/wiki/HowItWorks#starting-an-annotated-activity
Before
image

After
image


Also OnActivityResult page gave example like this.

@OnActivityResult(REQUEST_CODE)
void onResult(int resultCode, Intent data, @Extra String value) {
}

extra noo applicable to parameter

But it didn't work. So I change this @Extra -> @OnActivityResult.Extra

@OnActivityResult(REQUEST_CODE)
void onResult(int resultCode, Intent data, @OnActivityResult.Extra String value) {
}

https://github.com/JuranoSaurus/androidannotations/wiki/OnActivityResult

Activity

  1. Francis0121 commented on Oct 8, 2015

    @Francis0121
    ContributorAuthor

    And I found @Extrain javadoc. And I update OnActivityResult.Extra in javadoc.
    Could i ask you may i create new pull request?

    https://github.com/JuranoSaurus/androidannotations/tree/1580_onActivityResult_javadoc_update

  2. WonderCsabo commented on Oct 8, 2015

    @WonderCsabo
    Member
  3. WonderCsabo commented on Oct 8, 2015

    @WonderCsabo
    Member
  4. Francis0121 commented on Oct 8, 2015

    @Francis0121
    ContributorAuthor

    Oh i'm sorry...
    I know that .. but i copied and pasted your commit message.. i missed it

    And I modified wiki page like this :
    Note that you can use the @OnActivityResult annotation to get result code and extra value.

  5. WonderCsabo commented on Oct 8, 2015

    @WonderCsabo
    Member

    I squashed two of your commits, so you may have to hard reset your wiki repo instead of merging:

    git fetch upstream
    git reset upstream/master --hard

    But if you have changes do not do that, commit them and rebase your branch to upstream/master.

    Thanks for these fixes to our doc, it really needs concern. Keep it up! :)

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions