Skip to content

Conversation

@sergeibbb
Copy link
Member

@sergeibbb sergeibbb commented Jun 18, 2024

Description

Looks like the docs indicate that you need to send an access token in the body for this request. I don't know if it's always been that way or if that's recent, but the route was previously working for us without doing this. We will need to update it (GLVSC-569)

Checklist

  • I have followed the guidelines in the Contributing document
  • My changes follow the coding style of this project
  • My changes build without any errors or warnings
  • My changes have been formatted and linted
  • My changes include any required corresponding changes to the documentation (including CHANGELOG.md and README.md)
  • My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

@axosoft-ramint axosoft-ramint self-requested a review June 18, 2024 17:17
@axosoft-ramint axosoft-ramint self-assigned this Jun 18, 2024
Copy link
Contributor

@axosoft-ramint axosoft-ramint left a comment

Choose a reason for hiding this comment

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

Looks good - just some minor changes.

@sergeibbb sergeibbb force-pushed the feature/GLVSC-569-fix-refresh branch from 0457eb7 to 41caf32 Compare June 19, 2024 08:13
Copy link
Contributor

@axosoft-ramint axosoft-ramint left a comment

Choose a reason for hiding this comment

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

LGTM

@axosoft-ramint axosoft-ramint merged commit da47858 into main Jun 19, 2024
@axosoft-ramint axosoft-ramint deleted the feature/GLVSC-569-fix-refresh branch June 19, 2024 18:12
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