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

Conversation

@dodgex
Copy link
Member

@dodgex dodgex commented Nov 11, 2014

implements issue #1235
based on PR #1234

do not merge before 4.0

@dodgex
Copy link
Member Author

dodgex commented Nov 11, 2014

i decided to use actions instead of value to match @Receiver

@WonderCsabo WonderCsabo added this to the 4.0 milestone Nov 11, 2014
@WonderCsabo
Copy link
Member

You forgot to update the test classes.

@dodgex
Copy link
Member Author

dodgex commented Nov 22, 2014

the rebase broke the code. its fixed now

@WonderCsabo
Copy link
Member

Thanks. Will merge to 4.0, but we may have to do several rebases until that, sorry.

@dodgex
Copy link
Member Author

dodgex commented Nov 22, 2014

no problem. :)

@WonderCsabo
Copy link
Member

Please rebase this PR so it can goo green.

@dodgex
Copy link
Member Author

dodgex commented Dec 9, 2014

not sure if you got notified about the rebase... so just a smal ping @WonderCsabo :)

@dodgex
Copy link
Member Author

dodgex commented Dec 9, 2014

a damn... i just realized that this PR is for 4.0. you just wanted the rebase to fix the build. /sorry

@WonderCsabo
Copy link
Member

You do not have to be sorry at all. Thanks for notifying and for the rebase.

yDelouis added a commit that referenced this pull request May 24, 2015
…tion

1235 do not use methodname as action
@yDelouis yDelouis merged commit 587c8cf into androidannotations:develop May 24, 2015
@yDelouis
Copy link
Contributor

Thanks

@dodgex dodgex deleted the 1235_do_not_use_methodname_as_action branch May 24, 2015 13:37
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