Re: Presskit 10 review

Lists: pgsql-www
From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Presskit 10 review
Date: 2018-01-30 19:14:30
Message-ID: 624D09A6-6982-4122-88DF-4FD3B08C77C0@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-www

After <DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se> I noticed a few more
small things in the 10.0 presskit, and realized I never reviewed it when it was
published. Attached are a few follow-up patches fixing the markup and some
other details.

0001 - Rename OSX to macOS to match the rest of the site
0002 - Fix various markup errors
0003 - Use https:// rather than http:// for postgresql.org links
0004 - Fix incorrect attribute separator in tags

cheers ./daniel

Attachment Content-Type Size
0004-Replace-non-breaking-space-as-attribute-separator.patch application/octet-stream 21.5 KB
0003-Use-https-for-all-postgresql.org-links.patch application/octet-stream 1.5 KB
0002-Fix-incorrect-markup.patch application/octet-stream 12.3 KB
0001-Presskit-10-rename-OSX-to-macOS.patch application/octet-stream 12.6 KB

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Re: Presskit 10 review
Date: 2018-02-01 16:44:17
Message-ID: CABUevEwHb8DMMBL+8Pb5KDFiYQM1T5qkqE_piC7kamdc8E3P3Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-www

On Tue, Jan 30, 2018 at 8:14 PM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:

> After <DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se> I noticed a few more
> small things in the 10.0 presskit, and realized I never reviewed it when
> it was
> published. Attached are a few follow-up patches fixing the markup and some
> other details.
>
> 0001 - Rename OSX to macOS to match the rest of the site
> 0002 - Fix various markup errors
> 0003 - Use https:// rather than http:// for postgresql.org links
> 0004 - Fix incorrect attribute separator in tags
>

1 and 3 apply fine, and have been pushed.

2 and 4 both have conflicts. Did you create them off the wrong branch or
something?

--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>


From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Re: Presskit 10 review
Date: 2018-02-01 19:32:05
Message-ID: 0692C40B-157B-40FC-A793-B37C5E4751D4@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-www

> On 01 Feb 2018, at 17:44, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>
> On Tue, Jan 30, 2018 at 8:14 PM, Daniel Gustafsson <daniel(at)yesql(dot)se <mailto:daniel(at)yesql(dot)se>> wrote:
> After <DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se <mailto:DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se>> I noticed a few more
> small things in the 10.0 presskit, and realized I never reviewed it when it was
> published. Attached are a few follow-up patches fixing the markup and some
> other details.
>
> 0001 - Rename OSX to macOS to match the rest of the site
> 0002 - Fix various markup errors
> 0003 - Use https:// rather than http:// for postgresql.org <http://postgresql.org/> links
> 0004 - Fix incorrect attribute separator in tags
>
> 1 and 3 apply fine, and have been pushed.

Thanks!

> 2 and 4 both have conflicts. Did you create them off the wrong branch or something?

Ah, 2 applies with a two lines of fuzz, I guess git apply requires a non fuzzed
application. New versions of 2 and 4 on top of HEAD are attached.

cheers ./daniel

Attachment Content-Type Size
0002-Fix-incorrect-markup-v2.patch application/octet-stream 12.6 KB
0004-Replace-non-breaking-space-as-attribute-separator-v2.patch application/octet-stream 21.5 KB

From: "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Re: Presskit 10 review
Date: 2018-02-01 19:38:23
Message-ID: DB66D98C-42B3-4F64-9313-95D194C0017E@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-www


> On Feb 1, 2018, at 2:32 PM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
>> On 01 Feb 2018, at 17:44, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>
>> On Tue, Jan 30, 2018 at 8:14 PM, Daniel Gustafsson <daniel(at)yesql(dot)se <mailto:daniel(at)yesql(dot)se>> wrote:
>> After <DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se <mailto:DE5BEE53-FC89-4E15-BB09-43E96E679479(at)yesql(dot)se>> I noticed a few more
>> small things in the 10.0 presskit, and realized I never reviewed it when it was
>> published. Attached are a few follow-up patches fixing the markup and some
>> other details.
>>
>> 0001 - Rename OSX to macOS to match the rest of the site
>> 0002 - Fix various markup errors
>> 0003 - Use https:// rather than http:// for postgresql.org <http://postgresql.org/> links
>> 0004 - Fix incorrect attribute separator in tags
>>
>> 1 and 3 apply fine, and have been pushed.
>
> Thanks!
>
>> 2 and 4 both have conflicts. Did you create them off the wrong branch or something?
>
> Ah, 2 applies with a two lines of fuzz, I guess git apply requires a non fuzzed
> application. New versions of 2 and 4 on top of HEAD are attached.
>
> cheers ./daniel
>
> <0002-Fix-incorrect-markup-v2.patch><0004-Replace-non-breaking-space-as-attribute-separator-v2.patch>

Thanks for fixing this. It should help make putting together the PG11 presskit easier.

Jonathan