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.

Add @Extra to @Receiver#1181

Merged
yDelouis merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:1180_extras_for_receiverCopy head branch name to clipboard
Oct 12, 2014
Merged

Add @Extra to @Receiver#1181
yDelouis merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:1180_extras_for_receiverCopy head branch name to clipboard

Conversation

@dodgex

@dodgex dodgex commented Oct 5, 2014

Copy link
Copy Markdown
Member

This PR adds an @Extra annotation for @Receiver enhanced methods.

@yDelouis: i modified the ExtraParameterHandler to accept a JDefinedClass. This allows to create the static fields for the extra key to be generated inside of the anonymous BroadcastReceiver class. could you review if it is ok how i did it?

@WonderCsabo

Copy link
Copy Markdown
Member

Please do not use vague commit messages as "update test". Explain why do you "update test". :)

@dodgex

dodgex commented Oct 5, 2014

Copy link
Copy Markdown
Member Author

@WonderCsabo updated the commit message ;)

@dodgex

dodgex commented Oct 5, 2014

Copy link
Copy Markdown
Member Author

wiki is next

@WonderCsabo

Copy link
Copy Markdown
Member

Please use @Receiver.Extra in all commit messages, since @Extra is ambiguous.

@dodgex

dodgex commented Oct 5, 2014

Copy link
Copy Markdown
Member Author

@dodgex

dodgex commented Oct 5, 2014

Copy link
Copy Markdown
Member Author

commit messages updated.

@WonderCsabo

Copy link
Copy Markdown
Member

I quickly reviewed this and i think this is OK, but i let @yDelouis do the final decisions since he just created the model for these kind of annotations.

@yDelouis yDelouis merged commit cc524ac into androidannotations:develop Oct 12, 2014
@yDelouis yDelouis added this to the 3.2 milestone Oct 12, 2014
@yDelouis

Copy link
Copy Markdown
Contributor

Thanks !

@yDelouis

Copy link
Copy Markdown
Contributor

Related to #1180.

@WonderCsabo

Copy link
Copy Markdown
Member

I merged your doc, @dodgex. Thanks!

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.

3 participants

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