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.

Sign up for GitHub

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

Jump to bottom

Use API for bypass-checks #3629

Merged
merged 10 commits into from Oct 9, 2020
Merged

Use API for bypass-checks #3629

Changes from 8 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
39e9997
Use api for `bypass-checks`
yakov116 Oct 7, 2020
c1dc13f
getRepoURL()
yakov116 Oct 7, 2020
9509fa3
YO! Will ya stop hardcoding github.com!!?!
yakov116 Oct 8, 2020
4e84493
Update source/features/bypass-checks.tsx
yakov116 Oct 8, 2020
ba66ba2
Update source/features/bypass-checks.tsx
yakov116 Oct 8, 2020
01b0f0f
Apply suggestions from code review
yakov116 Oct 8, 2020
80cfa4e
Fix
yakov116 Oct 8, 2020
15b16e1
Inline
yakov116 Oct 8, 2020
183e2a5
Remove unnecessary number
yakov116 Oct 9, 2020
173f6e4
Lint
yakov116 Oct 9, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
14 changes: 5 additions & 9 deletions source/features/bypass-checks.tsx
View file
Open in desktop
Expand Up @@ -3,17 +3,13 @@ import {observe} from 'selector-observer';
import * as pageDetect from 'github-url-detection';

import features from '.';
import fetchDom from '../helpers/fetch-dom';
import * as api from '../github-helpers/api';
import {getRepoURL} from '../github-helpers';

async function bypass(detailsLink: HTMLAnchorElement): Promise<void> {
const directLink = await fetchDom<HTMLAnchorElement>(
detailsLink.href,
'[data-hydro-click*="check_suite.external_click"]'
);

if (directLink) {
detailsLink.href = directLink.href;
}
const runId = new URLSearchParams(detailsLink.search).get('check_run_id') ?? detailsLink.pathname.split('/').pop();
const directLink = await api.v3(`repos/${getRepoURL()}/check-runs/${Number(runId)}`);
Copy link
Member

Choose a reason for hiding this comment

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

Why number? It goes into a string.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sorry I have no idea why I did that!

detailsLink.href = directLink.details_url;
}

function init(): void {
Expand Down