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 a raw flag to the completed command #188

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dwainm
Copy link
Collaborator

@dwainm dwainm commented Jul 14, 2022

Fixes #182

This command outputs the raw markdown one can see when editing a task.

To test:

  1. Create a task that has some markdown in it. Example:
[markdown link](https://github.com/)
  1. Mark that task as completed.

  2. From the project root run go run . sync

  3. From the project root run go run . cl and notice that you'll only see markdown link

  4. From the project root run go run . cl --raw and notice the markdown shows, exposing the link to github.com.

@dwainm
Copy link
Collaborator Author

dwainm commented Mar 3, 2023

@sachaos thinking about this more, would this work, and if so, should the flag be --raw or --no-format or --raw-md

@sachaos
Copy link
Owner

sachaos commented Mar 3, 2023

@dwainm I think --raw is OK.

@kenliu
Copy link
Collaborator

kenliu commented Mar 15, 2023

I'll review this issue over the next few days.

@kenliu
Copy link
Collaborator

kenliu commented Mar 21, 2023

Hi @dwainm i'm not sure why, but when I checked out this PR and ran sync I consistently get this error message (403). Once I switch back to master it seems to work fine.

todoist on  dwainm/master:master [?] via 🐹 v1.20.1 
➜ ./todoist sync    
Error: bad request: 403 Forbidden

@kenliu
Copy link
Collaborator

kenliu commented Apr 29, 2023

hi @dwainm I think this PR needs to be rebased against master to verify that it is working (and there are conflicts right now). If it's okay with you I think (?) I can push to your PR but since your PR is on the master branch of your fork (instead of a branch off of master) that will change the state of your fork. Just a heads up in case I decide to do this.

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.

Allow raw text to be extracted
3 participants