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

Improvements to README.md. Also closes #4901 #4930 #4931

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yonas
Copy link

@yonas yonas commented Sep 7, 2024

@yonas yonas force-pushed the yonas-patch-readme-improvements branch from 28ca487 to cb98119 Compare September 7, 2024 12:51
@yonas yonas changed the title Improvements to README.md Improvements to README.md. Also closes #4901 #4930 Sep 7, 2024
Various improvements to the README.md
@yonas yonas force-pushed the yonas-patch-readme-improvements branch from 73250eb to 3eda1dd Compare September 7, 2024 16:45
[![Build](https://github.com/dani-garcia/vaultwarden/actions/workflows/build.yml/badge.svg)](https://github.com/dani-garcia/vaultwarden/actions/workflows/build.yml)
[![ghcr.io](https://img.shields.io/badge/ghcr.io-download-blue)](https://github.com/dani-garcia/vaultwarden/pkgs/container/vaultwarden)
[![ghcr.io Pulls](https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fipitio.github.io%2Fbackage%2Fdani-garcia%2Fvaultwarden%2Fvaultwarden.json&query=%24.downloads&label=ghcr.io%20pulls)](https://github.com/dani-garcia/vaultwarden/pkgs/container/vaultwarden)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not a fan of including changes from an active PR.

> [!TIP]
> [Please report any bugs or suggestions to us directly](#get-in-touch), regardless of whatever clients you are using (mobile, desktop, browser...).
>
> Do not use the official support channels.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the line break? The "official support channels" refer to Bitwarden support channels which is lost if you break up the paragraph like this. So please clarify what is meant by that or keep it in the same paragraph.


---

## Features

Basically full implementation of Bitwarden API is provided including:
A full implementation of the Bitwarden API:
Copy link
Contributor

Choose a reason for hiding this comment

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

Vaultwarden does not implement the full Bitwarden API. It notably is missing the Bitwarden Public API as stated in the wiki, so the qualifier should remain or better the sentence should be clarified what is meant.

* Authenticator and U2F support
* YubiKey and Duo support
* Organizations
* Attachments and send
Copy link
Contributor

Choose a reason for hiding this comment

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

Bitwarden Send is a product name so this should arguably stay capitalized.

* Organizations
* Attachments and send
* Vault API
* Serving static files
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the important part would be that the server comes bundled with a modified version of the Bitwarden web-vault not that Vaultwarden serves static files.


## Get in touch
## Get in touch ☕
### Discussions
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a fan of these sub headlines.

To ask a question, offer suggestions or new features or to get help configuring or installing the software, please use [GitHub Discussions](https://github.com/dani-garcia/vaultwarden/discussions) or [the forum](https://vaultwarden.discourse.group/).

### Issues
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably say "Bug reports" instead of "Issues" as common support questions and configuration issues should be asked in Github discussions or our Discourse forum (but like I said I'd rather not have these sub headlines in the first place.)

Comment on lines +12 to +14
Alternative implementation of the Bitwarden server API written in Rust and compatible with [upstream Bitwarden clients](https://bitwarden.com/download/).

Vaultwarden is the perfect self-hosted solution when running the official service is too resource-heavy.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think I'd prefer the description from the wiki. Maybe we should even include the sentence regarding the target audience.

Suggested change
Alternative implementation of the Bitwarden server API written in Rust and compatible with [upstream Bitwarden clients](https://bitwarden.com/download/).
Vaultwarden is the perfect self-hosted solution when running the official service is too resource-heavy.
Vaultwarden is an unofficial Bitwarden server implementation written in Rust that is targeted towards individuals, families and smaller organizations. It is compatible with the [official Bitwarden clients](https://bitwarden.com/download/).
Vaultwarden is ideal for self-hosted deployments where running the official resource-heavy service would be undesirable.


#### ⚠️**IMPORTANT**⚠️: When using this server, please report any bugs or suggestions to us directly (look at the bottom of this page for ways to get in touch), regardless of whatever clients you are using (mobile, desktop, browser...). DO NOT use the official support channels.
> [!TIP]
> [Please report any bugs or suggestions to us directly](#get-in-touch), regardless of whatever clients you are using (mobile, desktop, browser...).
Copy link
Contributor

Choose a reason for hiding this comment

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

The link does not work because you have added a space and an emoji below.


## Get in touch
## Get in touch ☕
Copy link
Contributor

Choose a reason for hiding this comment

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

The fragment identifier would change to #get-in-touch-, so we might not want to change the heading because of that.

@stefan0xC stefan0xC mentioned this pull request Sep 21, 2024
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