joplin / plugin-kanban

It allows notes in a notebook to be organized in a kanban board.
https://discourse.joplinapp.org/t/kanban-board-project/17469
132 stars 17 forks source link

Add "Close" button to close kanban and disable auto closing #44

Closed benlau closed 1 year ago

benlau commented 1 year ago

Implements #21

Screenshot 2023-06-22 at 10 04 30 AM

mhelleboid commented 1 year ago

This works as described.

The board lifecycle seems to have change a bit, but it seems ok.

Not sure that the initial feature request suggested to also disable the "auto closing". In this case this could be an option. Anyway, this can be done in an additional commit/branch.

benlau commented 1 year ago

@mhelleboid Thanks for reviewing!

yes, the auto closing is not described in the feature requirement.

However, as we now have a close button and therefore I think auto closing is not needed anymore as user could choose when should it be closed. It is more flexible.

Example:

  1. Edit a kanban board
  2. User find that a note should be added to the kanban
  3. Open the note and change folder/tag to add to kanban
  4. The board closed and user need to open the kanban again to resume the works but it may need to wait for a few seconds for reloading

Disable the auto closing feature could save the time for reloading in step 4

@PackElend , what do you think?

mhelleboid commented 1 year ago

Ok the scenario makes sense, and I agree that this is a good thing

PackElend commented 1 year ago

Sounds good but let me check the history why auto close exsist

benlau commented 1 year ago

@PackElend thx. If it is approved, I may try to work on a feature to drag a note from note list to a Kanban column directly.

PackElend commented 1 year ago

could find anything and still don't see a reason to keep auto close but others may think differently. I started a "survey", see https://discourse.joplinapp.org/t/shall-we-replace-auto-close-by-close-button/31440

PackElend commented 1 year ago

let's wait for a week, if it is not significant response I'll merge it

PackElend commented 1 year ago

I may try to work on a feature to drag a note from note list to a Kanban column directly.

@benlau please go ahead but open a topic in the forum as well, to allow the community to join the development :)

mhelleboid commented 1 year ago

The code and the PR is ok for me, from a technical and functional (auto close) point of view

PackElend commented 1 year ago

@benlau do you want to add some comments / do changes due to the latest review, if necessary?

So that I can merge it.

benlau commented 1 year ago

@PackElend I think it should be ready to merge. thx

AndiLeni commented 8 months ago

@benlau Sorry to comment on this old pr, but how do I activate this feature? The button is missing on my end.

benlau commented 8 months ago

@AndiLeni I think the change is not published yet.

@PackElend Do you have the permission to release a new version?

PackElend commented 8 months ago

Do you have the permission to release a new version?

will check

PackElend commented 5 months ago

@mhelleboid @benlau

Eventually GSoC is rolling smoothly and we are looking into other things. We get errors when we build it:

> joplin-plugin-kanban@1.0.7 prepare
> npm run dist

> joplin-plugin-kanban@1.0.7 dist
> webpack --joplin-plugin-config buildMain && webpack --joplin-plugin-config buildExtraScripts && webpack --joplin-plugin-config createArchive

do you get the same when you do that?

it is discussed in https://discord.com/channels/610762468960239627/919979500765335613/1240377195730440274

PackElend commented 5 months ago

https://github.com/joplin/plugin-kanban/pull/50 could fix it, let's see.