not perfect but in the right ideas #45

Merged
merged 14 commits into from Jun 7, 2012

Conversation

Projects
None yet
3 participants
@ghost

ghost commented May 9, 2012

I've made changes in your source about some troubles with url rewriting. I add i18n support on your app ^^.

Be carefull of this pull request. Test and read it plz ;)

Owner

Datawalke commented May 22, 2012

Thank you! I will review this get back to you shortly. Thanks again!

@ghost

ghost commented Jun 5, 2012

Other modifications in this pull request, your sourcecode was limited to root directory for installation, I updated the code to permit root or subfolders too.

Owner

Datawalke commented Jun 6, 2012

Thank you! I will review and get this added in. I really appreciate your modifications. I apologize for my delay in updates and will have you pull request an the others reviewed before the end of the week.

Thank you again,

  • Jim

On Jun 5, 2012, at 7:13 PM, cyberviraxreply@reply.github.com wrote:

Other modifications in this pull request, your sourcecode was limited to root directory for installation, I updated the code to permit root or subfolders too.


Reply to this email directly or view it on GitHub:
https://github.com/Datawalke/Coordino/pull/45#issuecomment-6139361

@Datawalke Datawalke added a commit that referenced this pull request Jun 7, 2012

@Datawalke Datawalke Merge pull request #45 from cybervirax/master
not perfect but in the right ideas
7791e7d

@Datawalke Datawalke merged commit 7791e7d into Datawalke:master Jun 7, 2012

stas commented Jun 7, 2012

Hey @Datawalke
could you please add tmp dir to gitignore.
With this merge, some logs got into master.

@cybervirax 👍 keep it up!

@gmponos gmponos pushed a commit to gmponos/Coordino that referenced this pull request Feb 2, 2016

@lorenzo lorenzo Merge pull request #45 from WyriHaximus/master
We're not using Configure here, why define it?
68a34fb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment