diff mbox series

[FFmpeg-devel] web: add news about IRC move (stolen from twitter)

Message ID 20210618141816.833-1-michael@niedermayer.cc
State New
Headers show
Series [FFmpeg-devel] web: add news about IRC move (stolen from twitter) | expand

Checks

Context Check Description
andriy/configure warning Failed to apply patch

Commit Message

Michael Niedermayer June 18, 2021, 2:18 p.m. UTC
Feel free to suggest better wording

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 src/index | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

James Almer June 18, 2021, 7:24 p.m. UTC | #1
On 6/18/2021 11:18 AM, Michael Niedermayer wrote:
> Feel free to suggest better wording
> 
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>   src/index | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/src/index b/src/index
> index e456ed2..3f2dfa5 100644
> --- a/src/index
> +++ b/src/index
> @@ -35,6 +35,13 @@
>       News
>     </h1>
>   
> +  <h3 id="irc2021">June 18th, 2021, IRC</h3>
> +  <p>
> +    We have a new IRC home at @liberachat
> +    now! Feel free to join us at #ffmpeg and #ffmpeg-devel. Unsure About channel? More info at <a href="https://ffmpeg.org/contact.html#IRCChannels">contact#IRCChannels</a>

Remove the @ character and use "Libera Chat" instead of liberachat since 
this is not Twitter, and remove the "Unsure About channel?" part as it's 
both broken an unnecessary.
Maybe also include a line stating that the project no longer has a 
presence in Freenode to make it clear it's not an alternative IRC server.

LGTM with those changes.

> +  </p>
> +
> +
>     <h3 id="pr4.4">April 8th, 2021, FFmpeg 4.4 "Rao"</h3>
>     <p>
>       <a href="download.html#release_4.4">FFmpeg 4.4 "Rao"</a>, a new
>
Michael Niedermayer June 19, 2021, 2:10 p.m. UTC | #2
On Fri, Jun 18, 2021 at 04:24:02PM -0300, James Almer wrote:
> On 6/18/2021 11:18 AM, Michael Niedermayer wrote:
> > Feel free to suggest better wording
> > 
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >   src/index | 7 +++++++
> >   1 file changed, 7 insertions(+)
> > 
> > diff --git a/src/index b/src/index
> > index e456ed2..3f2dfa5 100644
> > --- a/src/index
> > +++ b/src/index
> > @@ -35,6 +35,13 @@
> >       News
> >     </h1>
> > +  <h3 id="irc2021">June 18th, 2021, IRC</h3>
> > +  <p>
> > +    We have a new IRC home at @liberachat
> > +    now! Feel free to join us at #ffmpeg and #ffmpeg-devel. Unsure About channel? More info at <a href="https://ffmpeg.org/contact.html#IRCChannels">contact#IRCChannels</a>
> 
> Remove the @ character and use "Libera Chat" instead of liberachat since
> this is not Twitter, and remove the "Unsure About channel?" part as it's
> both broken an unnecessary.
> Maybe also include a line stating that the project no longer has a presence
> in Freenode to make it clear it's not an alternative IRC server.

changes done and applied

thx

[...]
diff mbox series

Patch

diff --git a/src/index b/src/index
index e456ed2..3f2dfa5 100644
--- a/src/index
+++ b/src/index
@@ -35,6 +35,13 @@ 
     News
   </h1>
 
+  <h3 id="irc2021">June 18th, 2021, IRC</h3>
+  <p>
+    We have a new IRC home at @liberachat
+    now! Feel free to join us at #ffmpeg and #ffmpeg-devel. Unsure About channel? More info at <a href="https://ffmpeg.org/contact.html#IRCChannels">contact#IRCChannels</a>
+  </p>
+
+
   <h3 id="pr4.4">April 8th, 2021, FFmpeg 4.4 "Rao"</h3>
   <p>
     <a href="download.html#release_4.4">FFmpeg 4.4 "Rao"</a>, a new