public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Dylan Whyte <d.whyte@proxmox.com>
To: Dietmar Maurer <dietmar@proxmox.com>,
	Proxmox Backup Server development discussion
	<pbs-devel@lists.proxmox.com>
Subject: Re: [pbs-devel] [PATCH proxmox-backup 1/2] docs: language and formatting fixup
Date: Fri, 8 Oct 2021 09:42:11 +0200	[thread overview]
Message-ID: <56f10636-1a8b-2e20-66b5-94b0d26aba0f@proxmox.com> (raw)
In-Reply-To: <b0cf0835-e3e6-c1dc-4277-b03cdc0d8a64@proxmox.com>


On 10/8/21 6:59 AM, Dietmar Maurer wrote:
> some questions below:
>
> On 10/6/21 5:19 PM, Dylan Whyte wrote:
>> --- a/docs/installation.rst
>> +++ b/docs/installation.rst
>> @@ -19,24 +19,24 @@ for various management tasks such as disk 
>> management.
>>      `Proxmox Backup`_ without the server part.
>
> We use `Proxmox Backup`_ here (which is defined in epilog.rst)
>
>>   The disk image (ISO file) provided by Proxmox includes a complete 
>> Debian system
>> -as well as all necessary packages for the `Proxmox Backup`_ server.
>> +as well as all necessary packages for the `Proxmox Backup Server`_.
>>     The installer will guide you through the setup process and allow
>> -you to partition the local disk(s), apply basic system configurations
>> -(e.g. timezone, language, network), and install all required packages.
>> +you to partition the local disk(s), apply basic system configuration
>> +(for example timezone, language, network), and install all required 
>> packages.
>>   The provided ISO will get you started in just a few minutes, and is 
>> the
>>   recommended method for new and existing users.
>>   -Alternatively, `Proxmox Backup`_ server can be installed on top of an
>> +Alternatively, `Proxmox Backup Server`_ can be installed on top of an
>
> So why do you change that here? Where is `Proxmox Backup Server`_ 
> defined?

Ah that's my bad. I update the epilog.rst as I thought it made more 
sense to have the hyperlink cover the full name of the product, but i 
forgot to include it with the patch..

Will fix it in a v2.

>
>
Dylan




      reply	other threads:[~2021-10-08  7:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-06 15:19 Dylan Whyte
2021-10-06 15:19 ` [pbs-devel] [PATCH proxmox-backup 2/2] docs: Update for new features/functionality Dylan Whyte
2021-10-08  4:59 ` [pbs-devel] [PATCH proxmox-backup 1/2] docs: language and formatting fixup Dietmar Maurer
2021-10-08  7:42   ` Dylan Whyte [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56f10636-1a8b-2e20-66b5-94b0d26aba0f@proxmox.com \
    --to=d.whyte@proxmox.com \
    --cc=dietmar@proxmox.com \
    --cc=pbs-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal