-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial commit of new magical-parameters-feature #20
base: master
Are you sure you want to change the base?
Conversation
I'll test this week. Thanks for putting the time into this. |
Ok, tested. Works as advertised. Yes, you need to add checks for users who do not have perms in the project. I would also add a check for if someone appends the magical params instead of prepends, appending causes issues. Othwerwise, i think this is a great solve, and appreciate the work put into it. if you need assistance with the checks, or anything, just let me know. |
I am currently working on the ToDos ... will be back soon with an updated commit :-) |
Here we go :-) I think this should be working well.
Checks added - If "magic" user has no access .. subtask is assigned to default assignee
Added that too
Thanks New helpers I have already prepared a local branch, where I reduced the need to register different actions for vanilla / Subtaskdate, but rather also handle the different needs, by using conditional parameter-assignments assisted by a helper-method. Looking forward to hear what you think |
@manne65-hd I can't test today, I think someone at home unplugged my server, but will test next chance i get. |
@manne65-hd |
Hey Craig, Does it mean I will have commit access to the main repository ?? O.K. Found it now ... Challenge accepted :-) |
yes, it means you can commit directly to the repo, merge pull requests, close issues, make releases, etc...you have full access. |
O.K. merged my branch that removes the last bits of separate "vanilla"-files Do you want me to merge this into the base branch or should I wait until you have tested? |
Depends on your confidence level. ;) Do you know how to make a release and update the kanboard/website? |
@@ -40,7 +39,7 @@ public function getPluginVersion() | |||
{ | |||
return '2.1.0'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
up version to 2.2.0
I am pretty tired now, so I'd better wait until tomorow :-)
Yes .. I already did that with my first plugin Which sort of Merge pull would you prefer ( merge commit / squash / rebase ) ? As this is the first commit of a whole new feature I'd go for squashing it into one commit. What do you think? |
I always just do merge, never done the others to offer advice. |
I am facing some challenges with the Do not duplicate subtasks-option, because with the introduction of the magical parameters this has gotten a bit more "tricky". I am working on it, and am confident to find a solution :-) Just wanted to let you know, that I am still working on this. |
You ever finish this? just noticed this got left unmerged. I have no idea where you left off. |
Hey Craig, Funny enough, I have just re-entered the "kanboard-world", because we will start making extended use of it in my company. All the best, |
Hey Craig,
I really like this plugin and funny enough, while testing it I had the same idea, that you have mentioned over here some time ago
So here's my approach to solve this (and even add some more features)
While retaining the current input-form, when creating AutoSubtasks I have added what I call "magical" parameters, that can be used on each line to overwrite any of these 2 or 3 parameters :
What are "magical" parameters?
These parameters can (optionally) be prepended to each line by wrappping them into curly braces:
Pictures are worth a thousand words
The new options are explained, when hovering the mouse over the INFO-icon ...
Task with matching category with the SubTasks created by the automatic action
ToDo
Looking forward to hear what you think of my idea.