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

Tagging the tasks #43

Merged
merged 5 commits into from
Feb 5, 2024
Merged

Tagging the tasks #43

merged 5 commits into from
Feb 5, 2024

Conversation

gzukowski
Copy link
Contributor

@gzukowski gzukowski commented Dec 29, 2023

Clickup task:
https://app.clickup.com/t/8693be6ue

What:
Added the source of the issue and changed the way of handling the tags.

Why:
There was a need for adding tags, which represents the source of the issue.

@gzukowski
Copy link
Contributor Author

There is a need for a token that has permission for adding tags (current devops account's permissions are too low).

Copy link
Contributor

@alicja-gruzdz alicja-gruzdz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is looking good, one comment about where to store the tags.

js/clickup-add-issues/clickupClient.ts Outdated Show resolved Hide resolved
@tomekcrm tomekcrm force-pushed the feat/task-with-tag branch 3 times, most recently from 57de876 to a55e5fa Compare January 3, 2024 15:37
Copy link
Contributor

@tomekcrm tomekcrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@a-tylenda a-tylenda merged commit db2a35b into main Feb 5, 2024
1 check passed
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.

4 participants