Skip to content

remove graphql api references from support bundle#320

Merged
divolgin merged 1 commit intomainfrom
divolgin/sc-106531/remove-graphql-api-references-from-support
Jun 27, 2024
Merged

remove graphql api references from support bundle#320
divolgin merged 1 commit intomainfrom
divolgin/sc-106531/remove-graphql-api-references-from-support

Conversation

@divolgin
Copy link
Contributor

@divolgin divolgin commented Jun 27, 2024 •
edited
Loading

  • removes watch-id parameter, which no longer works
  • removes customer-id parameter, which was deprecated and no longer works.
  • replaces graphql client with a REST client, and changes default endpoint to https://api.replicated.com/market

@divolgin divolgin force-pushed the divolgin/sc-106531/remove-graphql-api-references-from-support branch from 82eedc7 to 945e430 Compare June 27, 2024 15:31
@divolgin divolgin marked this pull request as draft June 27, 2024 15:34
@divolgin divolgin force-pushed the divolgin/sc-106531/remove-graphql-api-references-from-support branch 2 times, most recently from 93c11eb to 84835ba Compare June 27, 2024 15:52
@divolgin divolgin marked this pull request as ready for review June 27, 2024 15:57
@divolgin divolgin force-pushed the divolgin/sc-106531/remove-graphql-api-references-from-support branch 2 times, most recently from a92e782 to 1220de2 Compare June 27, 2024 16:06
laverya
laverya reviewed Jun 27, 2024
View reviewed changes
@divolgin
Remove support for Ship Watches
9e43ed3 
Remove graphql-api references

remove graphql queries
@divolgin divolgin force-pushed the divolgin/sc-106531/remove-graphql-api-references-from-support branch from 1220de2 to 9e43ed3 Compare June 27, 2024 17:47
@divolgin divolgin merged commit acb4bb2 into main Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@laverya laverya laverya approved these changes

Assignees

No one assigned

Labels

type::chore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@divolgin @laverya