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

Detect parent project of Todoist #1223

Closed
5000164 opened this issue Dec 12, 2018 · 2 comments · Fixed by #1224
Closed

Detect parent project of Todoist #1223

5000164 opened this issue Dec 12, 2018 · 2 comments · Fixed by #1224
Assignees

Comments

@5000164
Copy link

5000164 commented Dec 12, 2018

Environment details

  • Operation system: macOS Mojave
  • Extension version: 1.15.1
  • Browser: Google Chrome
  • App using Toggl Button: Todoist

Expected behavior

Detect parent project name when the same project name does not exist. (Previously behavior)

Actual behavior

Does not detect parent project name.

Additional details

The toggl-button behavior was changed by #1197 .
A Todoist project can be had hierarchy.
If a parent project is not detected, we need to prepare the same project names of Toggl as project names of Todoist.
However, it is difficult for me to create the same project names of Toggl as project names of Todoist because I often create and remove a subcategory.
For example, I often create and remove Feature subcategory when my category is Work > Client name > Project name > Feature name.

@tcrammond tcrammond self-assigned this Dec 12, 2018
@tcrammond
Copy link
Contributor

Hey @5000164, thanks for bringing this to our attention. We'll bring back the "nested project" detection. Sorry for the inconvenience in the meantime.

tcrammond added a commit that referenced this issue Dec 12, 2018
A previous release inadvertedly removed functionality from the Todoist integration. Project name is selected from the hierarchy of projects - projects in Todoist can be nested.

This partially reverts commit b3fbf2c.

The new feature for tags/labels is still intact.

Closes #1223.
@5000164
Copy link
Author

5000164 commented Dec 14, 2018

It's working in version 1.16.0.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants