-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat(keyAutoAdd): support key addition for GateHub #774
Conversation
Extension builds preview
|
865b607
to
0169d1b
Compare
0169d1b
to
861c6f5
Compare
width: '22rem', | ||
height: '8rem', | ||
width: '22em', | ||
height: '8em', | ||
fontSize: '16px', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GateHub has fontSize: 10px
on root, so our notification UI had size reduced. This fixes it.
return json.data as R; | ||
} | ||
|
||
export class GraphQlError extends Error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just for us to distinguish between regular error and gql error and (if needed) get error info.
Context
Closes #618
Changes proposed in this pull request
graphQl
fetch request helper.