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

[memgraph-2-17 < T782] Fix the Cypher examples for ZonedDateTime #782

Merged
merged 1 commit into from
May 21, 2024

Conversation

antepusic
Copy link
Contributor

Description

This PR corrects the Cypher examples for ZonedDateTime so that they construct valid values.

Pull request type

  • Fix or improvement of an existing page

Related PRs and issues

PR this doc page is related to: #1866 (corrects the docs written for it)

Checklist:

  • Check all content with Grammarly
  • Perform a self-review of my code
  • Make corresponding changes to the rest of the documentation (consult with the DX team)
  • The build passes locally
  • My changes generate no new warnings or errors
  • Add a corresponding label
  • If release-related, add a product and version label
  • If release-related, add release note on product PR N/A (correction)

@antepusic antepusic added priority: high (error) A mistake in the documentation, or information that is crucial for a correct flow status: ready PR is ready for review labels May 21, 2024
@antepusic antepusic self-assigned this May 21, 2024
@antepusic antepusic requested a review from kgolubic as a code owner May 21, 2024 15:15
Copy link

vercel bot commented May 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2024 3:16pm

@antepusic antepusic changed the title [memgraph-2-17] Fix the Cypher examples for ZonedDateTime [memgraph-2-17 < T782] Fix the Cypher examples for ZonedDateTime May 21, 2024
Copy link
Collaborator

@kgolubic kgolubic left a comment

Choose a reason for hiding this comment

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

Everything looks OK.

@kgolubic kgolubic merged commit 7ace6d5 into memgraph-2-17 May 21, 2024
2 checks passed
@kgolubic kgolubic deleted the fix-zoned-date-time-examples branch May 21, 2024 15:31
@antepusic antepusic removed the status: ready PR is ready for review label May 21, 2024
kgolubic added a commit that referenced this pull request May 23, 2024
* Memgraph 2.17 docs

* Docs

* Zoned datetime data type (#723)

* Document how zoned datetime is supported

* Fix formatting

* Update the Differences in Cypher implementation page

* Update min/max functions

* Revert "Zoned datetime data type (#723)" (#763)

This reverts commit 16c8157.

* [Memgraph 2.17 < ]Update Kubernetes docs  (#773)

* Init docs for kubernetes.

* Fixes.

* Update page.

* Update.

* Fix.

* Update pages/getting-started/install-memgraph/kubernetes.mdx

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

* Update kubernetes.mdx

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* [memgraph-2.17 < ] --restore-replication-state-startup default to true (#762)

* Replication changes

* Update config docs

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>

* Memgrpah 2.17 release notes (#679)

* Initial commit

* Update release notes

* Update release notes

* Update styling

* Fix typo

* Update RN

* Update release notes

* Update RN

* Update Release notes

* Add HA changes for 2.17 (#749)

* add '#1932 discard request leadership on follower' changes from megraph

* add user other user actions from '#1981' PR

* add part for env flags

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>

* new: Add Lab 2.14 release notes (#779)

Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Fix the Cypher examples for ZonedDateTime (#782)

* Create if not exists 2.17 (#780)

* Added description for if not exists for users and roles

* Added changes to description for if not exists for users and roles

* Update role-based-access-control.mdx

* Update users.mdx

---------

Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Update release notes

* Update RN

* [memgraph-2-17 < ] Debugging memgraph (#765)

* Debugging memgraph

* Updated documentation

* Add Docker Compose

* Add one more todo

* Grammarly

* Fix image path

* Move to new location

* Update help-center.md

* Fix typo (#783)

* Debugging image

* Remove TODO mark

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Update release notes

* Update direct download  links

* Bolt routing (#777)

* Update RN

* Update RN

* Add bolt server to the output (#728)

* Update RN

* Update RN

* Data recovery on startup (#793)

* Add flags

* Update flags and remove callout

* Add callout

* Update RN

* Update release notes

* Update values for Vertex (#791)

* Update

* Update Delta values

* Fix typo

* Update code

* Update values

* Update one

* Update calculations

---------

Co-authored-by: Ante Pušić <ante.pusic@memgraph.io>
Co-authored-by: Ante Javor <ante.javor@memgraph.io>
Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Andi <andi8647@gmail.com>
Co-authored-by: Antonio Filipovic <61245998+antoniofilipovic@users.noreply.github.com>
Co-authored-by: Toni <toni.lastre@memgraph.io>
Co-authored-by: tonijurjevic96 <168409767+tonijurjevic96@users.noreply.github.com>
Co-authored-by: Josipmrden <josip.mrden@memgraph.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high (error) A mistake in the documentation, or information that is crucial for a correct flow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants