Skip to content
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

Add done checkbox to parent issue page #230

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

daniell
Copy link
Contributor

@daniell daniell commented May 22, 2015

Add a 'Done' checkbox to the decision list and parent item pages that will mark, or unmark an action item as Done.

aliceh75 and others added 16 commits May 21, 2015 11:49
…mplemented as a checkbox within a link. Clicking on it calls a view on the server, then reloads the existing page with the change applied. This makes it possible to have such action happen on clicking a tick box (rather than then having to press submit) without relying on javascript. So far this approach was only used for the 'watch' box. As we are introducing the same functionality for the 'done' checkbox, there could be a need for some refactoring. This is an experiment at finding a small refactoring, which formalizes this approach without trying to change the way it's implemented. As such it names such 'checkbox' action Single Action Views, and moves them in a file of their own, with their own base class. It may be that this is not the best approach, and either no refactoring, or rethinking the way this is implemented is a more adequate course of action.
Refactor the way the 'watch' checkbox works. On the UI side this is i…
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