-
Notifications
You must be signed in to change notification settings - Fork 805
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
chore: remake repo main README.md #2016
Conversation
@4F24L is attempting to deploy a commit to the KeepHQ Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
Thank you @4F24L! I’ll review it ASAP and suggest further edits |
Excited to see my first PR getting merged. |
@4F24L Thank you so much for this! I have a few points I would like you to address. Besides the "styling" of the page itself, the information is also out dated and doesn't send out the Keep "message" perfectly. I would suggest opening https://www.keephq.dev and trying to align the message there to what we currently have in README. The images are great but some text needs to be adjusted. Would you like to give it another try? |
Okay, I will try to make changes as per your suggestion. |
@4F24L I'm not sure about this issue, since the README is kinda "Keep's face" and I feel should be made by the core maintainers with the most context of what Keep does. |
For example, we want to add actual examples and videos/screenshots from the platform. |
I added the actual images and screenshots from the platform. please check that out. |
I checked that website, and I'm confused about that. Should I make changes to the landing page also?? or the data from the website should be updated on the readme file. |
I made some little changes.
review this ! |
Closes #1809
📑 Description
Key Changes:
Would you like to make any additional tweaks or adjustments to this redesign?
✅ Checks
ℹ Additional Information