Skip to content
This repository was archived by the owner on Mar 12, 2020. It is now read-only.

Adding some features - #22

Open
HanXHX wants to merge 3 commits into
boxuk:masterfrom
HanXHX:master
Open

Adding some features#22
HanXHX wants to merge 3 commits into
boxuk:masterfrom
HanXHX:master

Conversation

@HanXHX

@HanXHX HanXHX commented May 13, 2014

Copy link
Copy Markdown

Hi

There is my pull request. I deleted some useless files (not needed) and i added dynamic columns + suburi management.

Cheers!

Emilien

@gavD

gavD commented May 14, 2014

Copy link
Copy Markdown

Looks good to me! I no longer work at Box UK however so I can't test/merge this. Thanks for getting involved though!

@HanXHX

HanXHX commented May 14, 2014

Copy link
Copy Markdown
Author

OK I understand... Can you give me ownership about this repository or can you define a "true" licence?

@gavD

gavD commented May 14, 2014

Copy link
Copy Markdown

I suppose this would still be the canonical repository, perhaps somebody from Box UK could weigh in? If not, feel free to fork :-) @bendavies @h00zf1sh @jenkoian @lukequinnell ?

@jenkoian

Copy link
Copy Markdown
Contributor

Hi @HanXHX - thanks for the pull request, the changes look great! I haven't had chance to test out the changes, but am hoping to give it a quick run through tomorrow and then hopefully merge.

Cheers for chipping in @gavD 👍

Comment thread src/kanbanise.js

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe leave the default as just '/'

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yeah it's just an error... I use a sub-uri at work.

@jenkoian

Copy link
Copy Markdown
Contributor

Finally got a chance to have a proper look at this, sorry it took so long. Few minor things, but generally looks good and am happy to merge.

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