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.

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

[CORL-1202] GraphiQL Bug #3043

Merged
merged 2 commits into from Jul 24, 2020
Merged

[CORL-1202] GraphiQL Bug #3043

merged 2 commits into from Jul 24, 2020

Conversation

wyattjoh
Copy link
Collaborator

What does this PR do?

Fixes a bug with a broken CDN URL.

Waiting on a fix: graphql/graphql-playground#1238

What changes to the GraphQL/Database Schema does this PR introduce?

None.

How do I test this PR?

Load /graphiql and see it work!

@wyattjoh wyattjoh added the bug label Jul 24, 2020
@wyattjoh wyattjoh added this to the v6.4.0 milestone Jul 24, 2020
@wyattjoh wyattjoh requested a review from tessalt July 24, 2020 16:07
@wyattjoh wyattjoh added the 🚀 merge it! Pull requests that should be merged after status checks pass with a review label Jul 24, 2020
@kodiakhq kodiakhq bot merged commit 9aa4591 into master Jul 24, 2020
@kodiakhq kodiakhq bot deleted the bugfix/CORL-1202 branch July 24, 2020 17:19
@kgardnr kgardnr modified the milestones: v6.4.0, v6.3.1 Jul 29, 2020
kgardnr pushed a commit that referenced this pull request Jul 29, 2020
* fix logic around repeat posts with or without media embeds: (#3042)

* fix: restore ReportCommentEvent (#3037)

Co-authored-by: Wyatt Johnson <wyattjoh@gmail.com>

* fix: fixed issue with graphiql (#3043)

* README.md Update (#3045)

* chore: update README.md

* fix: removed README from doctoc

* fix: replaced emoji codes with emjoi

* fix: removed emoji's due to browser rendering issues

* fix: fixed bugs with install styles

Co-authored-by: Tessa Thornton <tessathornton@gmail.com>
Co-authored-by: Vinh <vinh@vinh.tech>
cvle added a commit that referenced this pull request Jul 30, 2020
* fix: various ui fixes

* fix: tooltip button active color

* fix: teal color

* fix: RTE borders

* fix: snapshots

* fix: reduce padding of show more of this conversation button

* Fix font weight fallback values

* Replace RTE validation messages with v3 CallOuts

CORL-1213

* More styling classnames

* chore: remove not used style

* fix: reaction button primary color

* fix snapshots

* harmonize data-variant on buttons

* [CORL-1215] Install Style Tweaks (#3046)

* fix logic around repeat posts with or without media embeds: (#3042)

* fix: restore ReportCommentEvent (#3037)

Co-authored-by: Wyatt Johnson <wyattjoh@gmail.com>

* fix: fixed issue with graphiql (#3043)

* README.md Update (#3045)

* chore: update README.md

* fix: removed README from doctoc

* fix: replaced emoji codes with emjoi

* fix: removed emoji's due to browser rendering issues

* fix: fixed bugs with install styles

Co-authored-by: Tessa Thornton <tessathornton@gmail.com>
Co-authored-by: Vinh <vinh@vinh.tech>

* fix: remove lineheight css variables and make them relative

* fix: adjust line-heights

* fix: update snapshots

* More css bindings for client styling

Set bindable class on community guidelines
Set bindable classname on "Comment on" label

* fix: more tweaks

* fix: snapshots

* fix: lint and tests

Co-authored-by: nick-funk <nick.funk@outlook.com>
Co-authored-by: Wyatt Johnson <wyattjoh@gmail.com>
Co-authored-by: Tessa Thornton <tessathornton@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🚀 merge it! Pull requests that should be merged after status checks pass with a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants