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

feat: Update the about page to include a brief history of the project. #18

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

Laphatize
Copy link
Contributor

Description

I included a brief history of the project in the /about section of the website.

Screenshot showcasing this:
Screenshot of Page

Issues Resolved

Addresses a point mentioned in #13.

Content: About Page (/about/) MVP: 1 paragraph that describes the history of the project

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
Copy link
Member

@stockholmux stockholmux left a comment

Choose a reason for hiding this comment

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

This is fantastic work @Laphatize!

I'll accept it with the changes suggested, but if you have time convert it from an HTML file to markdown and make each sentence one line.

about/index.html Outdated Show resolved Hide resolved
@Laphatize
Copy link
Contributor Author

Sure, no problem. I'll make those changes.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
@Laphatize Laphatize requested a review from stockholmux April 10, 2024 02:33
@Laphatize
Copy link
Contributor Author

Should be all taken care of. Sorry it took me so long - was swamped with homework.

Copy link
Member

@stockholmux stockholmux left a comment

Choose a reason for hiding this comment

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

LGTM

@stockholmux stockholmux merged commit 564804a into valkey-io:main Apr 10, 2024
1 check passed
@stockholmux
Copy link
Member

Congrats @Laphatize! Glad to have you as a contributor.

stockholmux added a commit that referenced this pull request Apr 10, 2024
* adds commands to top nav (#19)

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* Add the ability to pull in docs from `valkey-doc` (#22)

* adds support for pulling in valkey-docs

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* one more change to layout

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

---------

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* update code owners file (#23)

Add kyle to the code owners file

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* feat: Update the about page to include a brief history of the project. (#18)

* feat: Update the about page to include a brief history of the project.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

* feat: Use markdown instead of HTML & update wording of history

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>
Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
Co-authored-by: Pranav Ramesh <laphatize@protonmail.com>
AMoo-Miki pushed a commit to AMoo-Miki/valkey-website that referenced this pull request Apr 11, 2024
* adds commands to top nav (valkey-io#19)

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* Add the ability to pull in docs from `valkey-doc` (valkey-io#22)

* adds support for pulling in valkey-docs

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* one more change to layout

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

---------

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* update code owners file (valkey-io#23)

Add kyle to the code owners file

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>

* feat: Update the about page to include a brief history of the project. (valkey-io#18)

* feat: Update the about page to include a brief history of the project.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

* feat: Use markdown instead of HTML & update wording of history

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>
Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
Co-authored-by: Pranav Ramesh <laphatize@protonmail.com>
AMoo-Miki pushed a commit to AMoo-Miki/valkey-website that referenced this pull request Apr 11, 2024
valkey-io#18)

* feat: Update the about page to include a brief history of the project.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

* feat: Use markdown instead of HTML & update wording of history

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
(cherry picked from commit 564804a)
stockholmux pushed a commit that referenced this pull request Apr 11, 2024
#18)

* feat: Update the about page to include a brief history of the project.

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

* feat: Use markdown instead of HTML & update wording of history

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>

---------

Signed-off-by: Pranav Ramesh <laphatize@protonmail.com>
(cherry picked from commit 564804a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants