/sweepFile Feature Request: Leave a note artifact when sweeping tasks with /sweepFile #4

Activity

1. EduardMe commented on May 22, 2021

   

   EduardMe

   on May 22, 2021 via email

   Contributor

   More actions

   I think that’s a good place for feedback! I have added some minor changes today (mostly dialogs and options improved) and released it. The recent changes by @nmn have made it also far better to see what’s going on. Maybe similar to how project notes are opened one by one, opening daily notes and reporting what will be swept would help. There is also a forward option in the latest version I think. You can choose between moving and forwarding (copying).

   …

   On Sun 23. May 2021 at 00:41, David Wertheimer ***@***.***> wrote: I find this plugin super useful. It would be good for there to be some user feedback (telling the user what's been done). I also think it would be good to leave a note at the top of the note you're sweeping FROM, something like: [4 open tasks swept to 2020-05-22] I would also recommend at some point to add a command like /forwardTodos that doesn't sweep them out, but forwards them, leaving a copy in place in the original file with a >2020-05-22 at the end. Think about renaming "sweepFile", "moveTodos" to be more descriptive. sweepTodos might still be good for sweeping all files for todos. @nmn <https://github.com/nmn> @jgclark <https://github.com/jgclark> @EduardMe <https://github.com/EduardMe> If there's another place (e.g. Canny.io) I should put items like this, let me know. I just wanted @nmn <https://github.com/nmn> to see it. :) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#4>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZCAYXWEQIV6I42DP6FOWTTPAXK5ANCNFSM45LCF2HA> .
2. dwertheimer commented on May 23, 2021

   

   dwertheimer

   on May 23, 2021

   Last edited by dwertheimer

   CollaboratorAuthor

   More actions

   I love this user interaction. So cool. Nice job, @EduardMe.
   However, in testing this out, I think I found a bug in Editor.removeParagraph
   For instance, I put in 4 fake tasks and run the plugin. It does copy them all to my today note.
   But it does not cleanly delete them all.
   You will note from the pictures below, it deleted 2/3 of the lines, and removed just one letter from the line that remained!
   I added some logging and the plugin seems to be correctly calling Editor.removeParagraph each time with the right item.

   

   

   

   

   ...that's how it leaves the note, with one line still (mostly) there, even though removeParagraph was called for that para.
3. nmn commented on May 23, 2021

   

   nmn

   on May 23, 2021

   Last edited by nmn

   Collaborator

   More actions

   I wasn't able to reproduce the error you mention... Please see if you see the problem consistently, or maybe it was a one-off issue?

   On the feature front, as you see, I've already added an option to "Reschedule" instead of "Move" tasks within Daily Notes.
   Here are the rest of the features I envision:

   • Choose custom Target date instead of Today. (Was mostly waiting for a calendar picker, but I can use Input for now)
   • Ability to move tasks one at a time. (You'll get asked once for each note, and then you can choose to move or skip each task in that note)

   Are there any other features you would want?
4. dwertheimer commented on May 23, 2021

   

   dwertheimer

   on May 23, 2021 via email

   Last edited by dwertheimer

   CollaboratorAuthor

   More actions

   That's so strange. I can reproduce it consistently on 2 different computers, one running Big Sur, and one Catalina. Here's a screen record. https://www.dropbox.com/s/yhq35xm4ux71cuu/npsweep.mov?dl=0 https://user-images.githubusercontent.com/8949588/119280252-6026ea00-bbe5-11eb-8acb-b0d406f06400.mov

   👍React with 👍1Reacted by Naman Goel
5. nmn commented on May 24, 2021

   

   nmn

   on May 24, 2021

   Collaborator

   More actions

   I'll check on this soon. In the meantime I suggest you try Reschedule instead.
6. dwertheimer commented on May 24, 2021

   

   dwertheimer

   on May 24, 2021 via email

   CollaboratorAuthor

   More actions

   By the way I just tried it on the phone and it did the same thing. @eduard, It’s so cool that the same plug-ins run on the phone. I was not expecting that.

   …

   On Sun, May 23 2021 at 5:11 PM, Naman Goel ***@***.***> wrote: I'll check on this soon. In the meantime I suggest you try Reschedule instead. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#4 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACEI6VGQ5MIPIUIFIPAAXBTTPGKTFANCNFSM45LCF2HA> .
7. jgclark commented on May 24, 2021

   

   jgclark

   on May 24, 2021

   Collaborator

   More actions

   @dwertheimer FYI the plugin architecture was significantly restricted in the early days to be just JavaScript, not other languages, specifically to allow plugins to run on iOS as well.
8. nmn commented on May 25, 2021

   

   nmn

   on May 25, 2021

   Collaborator

   More actions

   @dwertheimer FYI the plugin architecture was significantly restricted in the early days to be just JavaScript, not other languages, specifically to allow plugins to run on iOS as well.

   I don't see this ever changing. Apple hasn't allowed arbitrary code on iOS that's not JS forever now.

   And we could allow some additional languages by providing tools to compile them to JS.
9. EduardMe commented on May 25, 2021

   

   EduardMe

   on May 25, 2021 via email

   Contributor

   More actions

   Jonathan also mentioned the removeParagraph issue today, I assume it's due to the changing lineIndex. Will have a look at that.

   …

   On Tue, May 25, 2021 at 2:53 AM Naman Goel ***@***.***> wrote: @dwertheimer <https://github.com/dwertheimer> FYI the plugin architecture was significantly restricted in the early days to be just JavaScript, not other languages, specifically to allow plugins to run on iOS as well. I don't see this ever changing. Apple hasn't allowed arbitrary code on iOS that's not JS forever now. And we could allow some additional languages by providing tools to compile them to JS. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#4 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZCAYVIKBPPJ2UBOB5TV5DTPLYI7ANCNFSM45LCF2HA> .
10. dwertheimer

    closed this as completedon Jul 4, 2021