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

ZIR-152: Automatically populate issues in linear form PRs #25

Merged
merged 2 commits into from
Sep 5, 2024
Merged

Conversation

shkoo
Copy link
Contributor

@shkoo shkoo commented Aug 29, 2024

No description provided.

@shkoo shkoo requested a review from mothran August 29, 2024 18:21
@github-actions github-actions bot changed the title Automatically populate issues in linear form PRs ZIR-152: Automatically populate issues in linear form PRs Aug 29, 2024
github-token: ${{ secrets.GITHUB_TOKEN }}
linear-api-key: ${{ secrets.LINEAR_API_KEY }}
linear-team-key: "ZIR"
linear-created-issue-state-id: "bab82211-2372-4199-89d3-71f1246cee71" # in-progress
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems weird to me that a state ID would look like a UUID... but as long as this means "in progress" and leaks nothing else, I guess that's fine. @mothran ?

Copy link
Contributor

Choose a reason for hiding this comment

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

... Wow. I just grepped through the HTML for the Linear board, and that's pretty cursed. But I guess LGTM.

Copy link

Choose a reason for hiding this comment

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

Yeah to get that either you have to use the API / graphql client OR do what I did and change the description of the state in the UI while you have chrome debug tools monitoring the network tab to see what the UUID it references on update is.

Copy link
Contributor

@jacobdweightman jacobdweightman left a comment

Choose a reason for hiding this comment

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

LGTM, but I'll defer to @mothran to approve

@shkoo shkoo merged commit 3abf1fd into main Sep 5, 2024
8 checks passed
@shkoo shkoo deleted the nils/linear branch September 5, 2024 17:11
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.

3 participants