Skip to content

DOC-493-Add Api links to SQL and Redis#525

Open
ghost wants to merge 14 commits into
mainfrom
DOC-493-Add-API-links-to-SQL-Redis
Open

DOC-493-Add Api links to SQL and Redis#525
ghost wants to merge 14 commits into
mainfrom
DOC-493-Add-API-links-to-SQL-Redis

Conversation

@ghost

@ghost ghost commented Dec 23, 2022

Copy link
Copy Markdown
  • Redis update for 17.12

Dino Lozina and others added 9 commits December 21, 2022 10:49
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
Co-authored-by: Diana Payton <105658419+diana-macrometa@users.noreply.github.com>
@ghost ghost added the DO NOT MERGE label Dec 23, 2022
@vercel

vercel Bot commented Dec 23, 2022

Copy link
Copy Markdown

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

Name Status Preview Updated
docs ❌ Failed (Inspect) Feb 23, 2023 at 5:25PM (UTC)

@diana-macrometa diana-macrometa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@jamesm-macrometa jamesm-macrometa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, but I would merge the two notes together so we don't have stacked note boxes.

Comment thread docs/queryworkers/redis/index.md
Comment thread docs/queryworkers/redis/index.md Outdated
Comment thread docs/queryworkers/redis/index.md Outdated
Comment thread docs/queryworkers/redis/index.md Outdated
@diana-macrometa diana-macrometa changed the base branch from main to 17-12-release February 13, 2023 15:07
@diana-macrometa

Copy link
Copy Markdown
Contributor

@dlozina-macrometa, please finish this PR

Dino Lozina and others added 4 commits February 23, 2023 18:22
Co-authored-by: James <86624594+jamesm-macrometa@users.noreply.github.com>
Co-authored-by: James <86624594+jamesm-macrometa@users.noreply.github.com>
Co-authored-by: James <86624594+jamesm-macrometa@users.noreply.github.com>
Co-authored-by: James <86624594+jamesm-macrometa@users.noreply.github.com>
@diana-macrometa

Copy link
Copy Markdown
Contributor

@dlozina-macrometa, how's this going?

Base automatically changed from 17-12-release to main April 17, 2023 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants