Skip to content

Issue #287 by Inlead: Rewrite ding campaign module.#16

Merged
kasperg merged 46 commits intoding2:masterfrom
easyddb:easyddb
Oct 29, 2015
Merged

Issue #287 by Inlead: Rewrite ding campaign module.#16
kasperg merged 46 commits intoding2:masterfrom
easyddb:easyddb

Conversation

@guddo
Copy link
Copy Markdown
Member

@guddo guddo commented Aug 7, 2015

Martin Cording and others added 30 commits March 13, 2015 14:27
Signed-off-by: Martin Cording <guddo@me.com>
Signed-off-by: Martin Cording <guddo@me.com>
Signed-off-by: Martin Cording <guddo@me.com>
Signed-off-by: Martin Cording <guddo@me.com>
Signed-off-by: Martin Cording <guddo@me.com>
Signed-off-by: Martin Cording <guddo@me.com>
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.

I would appreciate a comment describing why it is necessary to clear the cache at this point.

@kasperg
Copy link
Copy Markdown
Member

kasperg commented Oct 16, 2015

@guddo: I have reviewed this. In general things look great.

I have a request for an inline comment and a version should be adjusted as well.

@guddo guddo force-pushed the easyddb branch 2 times, most recently from 018bdbf to 1c65ef8 Compare October 28, 2015 12:43
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.

Please remove versioning information from the module.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry, this is a PR directly from our distribution - feel free to fork our repo or duplicate a PR like this.

@kasperg
Copy link
Copy Markdown
Member

kasperg commented Oct 29, 2015

Merged. Thanks.

kasperg added a commit that referenced this pull request Oct 29, 2015
Issue #287 by Inlead: Rewrite ding campaign module.
@kasperg kasperg merged commit b6dea97 into ding2:master Oct 29, 2015
@guddo
Copy link
Copy Markdown
Member Author

guddo commented Oct 29, 2015

@kasperg: See #18.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants