Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[master < E-util-docs] Write docs for util module #1012

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

mpintaric55334
Copy link
Contributor

Description

Please briefly explain the changes you made here.

This PR is for a documentation page of the util MAGE query module. The module contains a single function.

Pull request type

Please delete options that are not relevant and check the ones that are.

  • New documentation page

Related PRs and issues

PR this doc page is related to:

memgraph/mage#311

Closes (link to issue):

Checklist:

  • I checked all content with Grammarly
  • I performed a self-review of my code
  • I made corresponding changes to the rest of the documentation
  • The build passes locally
  • My changes generate no new warnings or errors

@mpintaric55334 mpintaric55334 added the status: ready PR is ready for review label Aug 29, 2023
@mpintaric55334 mpintaric55334 self-assigned this Aug 29, 2023
@mpintaric55334 mpintaric55334 changed the title Write docs for util module [master < E-util-docs] Write docs for util module Aug 30, 2023
</span>
);

The util module provides a set of handy functions for tasks like validation and generating cryptographic hash values. It's a valuable toolkit for simplifying various database-related operations.
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe it would be worthy to give an example of when this is valuable?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Honestly have no idea for what purpose the people who requested the function use it, could be password hashing, hash tables etc., so I will rephrase it differently

Copy link
Collaborator

@vpavicic vpavicic left a comment

Choose a reason for hiding this comment

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

I just left one comment

@vpavicic vpavicic added status: ship it PR approved and removed status: ready PR is ready for review labels Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants