Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Interpret @Extra annotated params as extras with @Receiver{Action}#1718

Merged
WonderCsabo merged 4 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:1717_receiverIntentAsExtraWonderCsabo/androidannotations:1717_receiverIntentAsExtraCopy head branch name to clipboard
Mar 4, 2016
Merged

Interpret @Extra annotated params as extras with @Receiver{Action}#1718
WonderCsabo merged 4 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:1717_receiverIntentAsExtraWonderCsabo/androidannotations:1717_receiverIntentAsExtraCopy head branch name to clipboard

Conversation

@WonderCsabo

Copy link
Copy Markdown
Member

Implements #1717 .

}
}

@Receiver(actions = "org.androidannotations.ACTION_1")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

here is an extra space :)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

also a string instead of a constant as in ReceiverWithActions. i think you did that to not polute this PR with extracting the other strings?

@dodgex

dodgex commented Mar 4, 2016

Copy link
Copy Markdown
Member

I think this looks ok just a little extra space that could be removed.

@WonderCsabo WonderCsabo force-pushed the 1717_receiverIntentAsExtra branch from f916ea1 to 50a27c5 Compare March 4, 2016 13:40
@WonderCsabo

Copy link
Copy Markdown
Member Author

@dodgex thanks, good points. Updated.

WonderCsabo added a commit that referenced this pull request Mar 4, 2016
Interpret @extra annotated params as extras with @receiver{Action}
@WonderCsabo WonderCsabo merged commit 2a38288 into androidannotations:develop Mar 4, 2016
@WonderCsabo WonderCsabo deleted the 1717_receiverIntentAsExtra branch March 4, 2016 14:01
@WonderCsabo WonderCsabo added this to the 4.0 milestone Mar 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.