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.

1235 do not use methodname as action#1236

Merged
yDelouis merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:1235_do_not_use_methodname_as_actionCopy head branch name to clipboard
May 24, 2015
Merged

1235 do not use methodname as action#1236
yDelouis merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:1235_do_not_use_methodname_as_actionCopy head branch name to clipboard

Conversation

@dodgex

@dodgex dodgex commented Nov 11, 2014

Copy link
Copy Markdown
Member

implements issue #1235
based on PR #1234

do not merge before 4.0

@dodgex

dodgex commented Nov 11, 2014

Copy link
Copy Markdown
Member Author

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
Copy Markdown
Member

You forgot to update the test classes.

@dodgex

dodgex commented Nov 22, 2014

Copy link
Copy Markdown
Member Author

the rebase broke the code. its fixed now

@WonderCsabo

Copy link
Copy Markdown
Member

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

@dodgex

dodgex commented Nov 22, 2014

Copy link
Copy Markdown
Member Author

no problem. :)

@WonderCsabo

Copy link
Copy Markdown
Member

Please rebase this PR so it can goo green.

@dodgex

dodgex commented Dec 9, 2014

Copy link
Copy Markdown
Member Author

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

@dodgex

dodgex commented Dec 9, 2014

Copy link
Copy Markdown
Member Author

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
Copy Markdown
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
Copy Markdown
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

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