Skip to content

Conversation

@SunsetTechuila
Copy link
Contributor

@SunsetTechuila SunsetTechuila commented Nov 21, 2025

I don't really expect this to be merged

Fixes #7968

Test URLs

#8785

Screenshot

image

@SunsetTechuila
Copy link
Contributor Author

Is there is a way to obtain pr subscription state via api?

@fregante fregante added the bug label Nov 21, 2025
@fregante
Copy link
Member

Is there is a way to obtain pr subscription state via api?

No, that's the issue 🥲

}

const body = {
query: 'dd170c659a085a45885ee5a168fc52c8',
Copy link
Member

Choose a reason for hiding this comment

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

I feel that this is a generated ID and it might break on the next deployment. Let's wait a week or two.

Copy link
Contributor Author

@SunsetTechuila SunsetTechuila Nov 21, 2025

Choose a reason for hiding this comment

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

I feel that this is a generated ID

Yes

Let's wait a week or two.

Absolutely

Copy link
Contributor Author

@SunsetTechuila SunsetTechuila Nov 21, 2025

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@SunsetTechuila SunsetTechuila Nov 29, 2025

Choose a reason for hiding this comment

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

Still works - the hash hasn't changed

Copy link
Contributor Author

@SunsetTechuila SunsetTechuila Dec 16, 2025

Choose a reason for hiding this comment

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

The hash has changed, but the server doesn't reject the request with an outdated hash

Co-authored-by: fregante <me@fregante.com>

async function addButtonIssue(subscriptionButton: HTMLButtonElement): Promise<void> {
const status = await getCurrentStatusIssue();
const previousRghButton = $optional('.rgh-status-subscription', subscriptionButton.parentElement!);
Copy link
Contributor Author

@SunsetTechuila SunsetTechuila Nov 21, 2025

Choose a reason for hiding this comment

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

Is there a better way to update the state?

const {id} = issue.repository.issue;

const body = {
query: 'd0752b2e49295017f67c84f21bfe41a3',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This hash seems to be quite stable

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

Labels

Development

Successfully merging this pull request may close these issues.

status-subscription missing from new issue view

2 participants