Skip to content

Set default shareEnabled to true #1035

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

zhammer
Copy link
Contributor

@zhammer zhammer commented Jun 9, 2019

It seems like this was accidentally introduced in some UI fixes (273697e#diff-de99fb113c90fc40c0f1349bddf05b09)
The PR (#642) that added shareEnabled set default to true

Fixes #983

Changes proposed in this pull request:

  • shareEnabled defaults to true
  • graphqlbin.com will have share button

- It seems like this was accidentally introduced in some UI fixes (graphql@273697e#diff-de99fb113c90fc40c0f1349bddf05b09)
- The PR (graphql#642) that added shareEnabled set default to true
- Resolves graphql#983
@Kikobeats
Copy link

@zhammer one question, could be possible to achieve this right now from settings object?

@qhenkart
Copy link

can this get a review and merged? It would be great to have a fix, or at the very least add this setting to the settings object

@zhammer
Copy link
Contributor Author

zhammer commented Sep 16, 2019

i'll try sharing in the prisma gitter chat

@lastmjs
Copy link

lastmjs commented Feb 1, 2020

Is it possible to enable sharing manually from a GraphQL playground?

@WrRaThY
Copy link

WrRaThY commented Jun 23, 2020

not being able to share from both the hosted website and local application defeats the whole purpose of graphqlbin.
I shouldn't have to host it myself (even locally) to be able to do that.
especially that this option is still marketed in the readme!

@CLAassistant
Copy link

CLAassistant commented Sep 23, 2020

CLA assistant check
All committers have signed the CLA.

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.

Unable to find share button
6 participants