-
Notifications
You must be signed in to change notification settings - Fork 139
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
draft: feat(import): add clickup import #220
base: master
Are you sure you want to change the base?
Conversation
|
Is this PR getting merged soon? We're interested in moving over from Clickup. |
Earlier I could not test the ClickUp importer. Right now, I'm busy at some other work. So no chance of it being merged anytime soon. |
Is there any updates, or do you intend to make some? I'd love to take over otherwise (will credit you as the starting point) |
Hey @Chew! |
@@ -1,38 +1,78 @@ | |||
{ | |||
"name": "linear", | |||
"private": true, | |||
"author": "Linear Orbit, Inc", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change on this line is not relevant to the PR, could keep it as it was previously.
"build:import": "lerna run build:import", | ||
"build:sdk": "lerna run build:sdk", | ||
"build:types": "npx tsc --project tsconfig.check.json", | ||
"build": "run-s generate build:sdk generate:test build:import build:types docs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Likewise for this line too. Not relevant to the proposed change in this PR.
"main": "dist/index-cjs.min.js", | ||
"module": "dist/index-es.min.js", | ||
"types": "dist/index.d.ts", | ||
"bin": { | ||
"linear-import": "bin/linear-import.js" | ||
}, | ||
"repository": "https://github.com/linear/linear", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a relevant code change
Medium: 3, | ||
Low: 4, | ||
}; | ||
return priorityMap[input] || 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! ⚡
No description provided.