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

Docs: remove useless code in code snippet #8990

Merged
merged 1 commit into from
Aug 4, 2023
Merged

Docs: remove useless code in code snippet #8990

merged 1 commit into from
Aug 4, 2023

Conversation

c-ciobanu
Copy link
Contributor

The paragraph above this code block says:
We're not going to let anyone update or delete a message, so we can remove those fields completely. Here's what the SDL file looks like after the changes:
Everything is removed from the code block except the UpdateContactInput that is not gonna be used by anything. This is just for the TS file, the JS file has the UpdateContactInput removed correctly.

@Tobbe
Copy link
Member

Tobbe commented Jul 30, 2023

Thanks for the fix 🙏

@Tobbe Tobbe added the release:docs This PR only updates docs label Jul 30, 2023
@Tobbe Tobbe added this to the next-release milestone Jul 30, 2023
@Tobbe Tobbe enabled auto-merge (squash) July 30, 2023 05:57
@jtoar jtoar disabled auto-merge August 4, 2023 18:54
@jtoar jtoar merged commit f3cde64 into redwoodjs:main Aug 4, 2023
27 of 29 checks passed
@c-ciobanu c-ciobanu deleted the patch-1 branch August 5, 2023 07:35
dac09 added a commit to dac09/redwood that referenced this pull request Aug 8, 2023
…nto try/apollo-ssr-stream

* 'fix/more-streaming-fixes' of github.com:dac09/redwood:
  Whoops
  Update packages/vite/src/utils.ts
  More suggestions, fix wrong path on reactRefresh script
  Use node path for consistency
  Fix portal head not rendering on the server
  docs(fonts): Update @font-face recommendation (redwoodjs#8986)
  Docs: remove useless code in code snippet (redwoodjs#8990)
  v6.0.3
  fix(router): Prevent rerendering authenticated routes on hash change (redwoodjs#9007)
  Remove the indexed type reference on AvailableRoutes (redwoodjs#8918)
  Remove debug logs
  Update packages/vite/src/streamHelpers.ts
dac09 added a commit to dac09/redwood that referenced this pull request Aug 9, 2023
…croll-reset

* 'main' of github.com:redwoodjs/redwood:
  fix(deps): update dependency pino to v8.15.0 (redwoodjs#9023)
  fix(deps): update dependency eslint to v8.46.0 (redwoodjs#9022)
  fix(deps): update dependency react-hook-form to v7.45.4 (redwoodjs#9017)
  chore(docs): reversion docs to include recent changes
  fix(deps): update dependency vite to v4.4.9 (redwoodjs#9018)
  v6.0.4
  fix(docs): update quick start to fix Storybook start up (redwoodjs#9014)
  cherry pick part of "fix: Improve GraphQL Schema Validation to allow Subscription types when not using Realtime and ensure schema does not use reserved names (redwoodjs#9005)"
  fix: Improve GraphQL Schema Validation to allow Subscription types when not using Realtime and ensure schema does not use reserved names (redwoodjs#9005)
  Docs: Explain the entry.client.{jsx,tsx} file (redwoodjs#8987)
  chore(deps): update dependency esbuild to v0.18.19 (redwoodjs#8983)
  chore(deps): update dependency nx-cloud to v16.2.0 (redwoodjs#8985)
  docs(fonts): Update @font-face recommendation (redwoodjs#8986)
  Docs: remove useless code in code snippet (redwoodjs#8990)
  v6.0.3
  fix(router): Prevent rerendering authenticated routes on hash change (redwoodjs#9007)
  Remove the indexed type reference on AvailableRoutes (redwoodjs#8918)
@jtoar jtoar modified the milestones: next-release, v6.1.0 Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:docs This PR only updates docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants