Skip to content

Add: local sistent package usage to Sistent About Page #6448

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 13 commits into
base: master
Choose a base branch
from

Conversation

M-DEV-1
Copy link
Contributor

@M-DEV-1 M-DEV-1 commented May 15, 2025

Description

This PR fixes #6402

Notes for Reviewers
Screencast from 2025-05-15 17-37-04.webm

Signed commits

  • Yes, I signed my commits.

Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
@M-DEV-1
Copy link
Contributor Author

M-DEV-1 commented May 15, 2025

Screencast.from.2025-05-15.17-37-04.webm

Copy link

github-actions bot commented May 15, 2025

@l5io
Copy link
Contributor

l5io commented May 15, 2025

🚀 Preview for commit 2694dbd at: https://6825ddc6ef25ddd5000c0c73--layer5.netlify.app

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

No link?

@vishalvivekm
Copy link
Contributor

@M-DEV-1
Thank you for your contribution!
Let's discuss this during the website call today at 7 AM CT | 5:30 PM IST.

Add it as an agenda item to the meeting minutes, if you would :)

@winkletinkle
Copy link

@vishalvivekm @M-DEV-1 was this discussed? What is the outcome? Why is there no clear path forward here?

@winkletinkle
Copy link

@leecalcote, I can update the text and get this job done, if needed.

@leecalcote
Copy link
Member

@winkletinkle, thank you. Hold that thought. I'm hopeful that either one of these other two contributors will finish the job that was started about a week without me having to follow up with them again.

Co-authored-by: Lee Calcote <leecalcote@gmail.com>
Signed-off-by: mahadevan <135952571+M-DEV-1@users.noreply.github.com>
@l5io
Copy link
Contributor

l5io commented May 20, 2025

🚀 Preview for commit bca98c7 at: https://682d0808b728c933db4a74e3--layer5.netlify.app

Co-authored-by: Lee Calcote <leecalcote@gmail.com>
Signed-off-by: Vivek Vishal <vishalvivek488@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
@l5io
Copy link
Contributor

l5io commented May 21, 2025

🚀 Preview for commit a0c4048 at: https://682da94b1cdba34d438aed6b--layer5.netlify.app

@vishalvivekm
Copy link
Contributor

vishalvivekm commented May 21, 2025

@vishalvivekm @M-DEV-1 was this discussed? What is the outcome? Why is there no clear path forward here?

Hi @winkletinkle, yes this was discussed and @M-DEV-1 was to accept suggestions and make any remaining updates.

@M-DEV-1
Copy link
Contributor Author

M-DEV-1 commented May 21, 2025

@vishalvivekm thank you for fixing the grammar, I've added instructions for npm link. I've added the do-not-merge label because I'm doing a few tests.

@l5io
Copy link
Contributor

l5io commented May 21, 2025

🚀 Preview for commit 0770362 at: https://682dac71300da6b056229f91--layer5.netlify.app

Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
@M-DEV-1
Copy link
Contributor Author

M-DEV-1 commented May 21, 2025

image
image
image
image

@l5io
Copy link
Contributor

l5io commented May 21, 2025

🚀 Preview for commit 8917182 at: https://682dbcfa72cfb874d5f151e2--layer5.netlify.app

@M-DEV-1
Copy link
Contributor Author

M-DEV-1 commented May 21, 2025

I've also updated the docs in the Sistent repo with npm link

@l5io
Copy link
Contributor

l5io commented May 26, 2025

🚀 Preview for commit 266a9e4 at: https://683441b98988c61794fc5d7c--layer5.netlify.app

@M-DEV-1 M-DEV-1 requested a review from vishalvivekm May 28, 2025 10:10
@l5io
Copy link
Contributor

l5io commented May 28, 2025

🚀 Preview for commit 0988ca4 at: https://6836e41ad77cc94680e851b5--layer5.netlify.app

@vr-varad
Copy link
Contributor

image
Screenshot from 2025-05-29 20-10-32

hey @M-DEV-1 mind remove the numbers (list)

  1. Method 1

doesn't make sense

M-DEV-1 and others added 4 commits May 30, 2025 15:52
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
…-DEV-1/layer5 into M-DEV-1/add-sistent-setup-instructions
@l5io
Copy link
Contributor

l5io commented May 30, 2025

🚀 Preview for commit 24c4a4b at: https://68398c8378f84c4c3d781048--layer5.netlify.app

@M-DEV-1
Copy link
Contributor Author

M-DEV-1 commented May 30, 2025

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.

Instructions for how to refer to your local Sistent fork
6 participants