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

Add Armeria Youtube videos #4416

Merged
merged 2 commits into from
Sep 7, 2022
Merged

Add Armeria Youtube videos #4416

merged 2 commits into from
Sep 7, 2022

Conversation

syleeeee
Copy link
Member

@syleeeee syleeeee commented Sep 6, 2022

We had Youtube live last April, and the site only showed the slides so far.
The LINE DevRel team posted the recorded videos, so I'd like to add them to the site as well.

Screenshot-armeria-site

<Article
title="인증 마이크로서비스 Armeria로 개발하기"
url="https://youtu.be/k1xh2rC_MbI"
author="김도한"
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks to @AmosDoan ❤️

<Article
title="라인 개발자들이 Spring 대신 Armeria 쓴 이유는?"
url="https://youtu.be/aoQO_bkYW94"
author="김기환, 임경수"
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks to @ghkim3221 and @ks-yim ❤️

Copy link
Contributor

@jrhee17 jrhee17 left a comment

Choose a reason for hiding this comment

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

Thanks @syleeeee ! 🙇 👍 🙇

@codecov
Copy link

codecov bot commented Sep 6, 2022

Codecov Report

Merging #4416 (5d25528) into master (5ff69a5) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master    #4416    +/-   ##
==========================================
  Coverage     73.93%   73.93%            
- Complexity    17826    17852    +26     
==========================================
  Files          1513     1516     +3     
  Lines         66300    66405   +105     
  Branches       8322     8347    +25     
==========================================
+ Hits          49016    49098    +82     
- Misses        13278    13293    +15     
- Partials       4006     4014     +8     
Impacted Files Coverage Δ
...ecorp/armeria/server/LengthBasedServiceNaming.java 75.00% <0.00%> (-16.67%) ⬇️
...m/linecorp/armeria/client/DecodedHttpResponse.java 85.00% <0.00%> (-5.00%) ⬇️
...rnal/common/stream/AbstractFixedStreamMessage.java 89.23% <0.00%> (-3.08%) ⬇️
...erver/graphql/protocol/AbstractGraphqlService.java 82.64% <0.00%> (-3.07%) ⬇️
...ia/internal/common/SplitHttpMessageSubscriber.java 80.45% <0.00%> (-3.01%) ⬇️
...inecorp/armeria/server/file/StreamingHttpFile.java 54.78% <0.00%> (-2.61%) ⬇️
.../linecorp/armeria/client/Http2ResponseDecoder.java 70.62% <0.00%> (-2.10%) ⬇️
...rmeria/internal/client/grpc/ArmeriaClientCall.java 80.24% <0.00%> (-1.24%) ⬇️
...inecorp/armeria/server/HttpResponseSubscriber.java 80.51% <0.00%> (ø)
...a/internal/server/annotation/AnnotatedService.java 88.00% <0.00%> (ø)
... and 14 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

title="Armeria를 소개합니다"
url="https://youtu.be/jlKb61lHOOw"
author="이희승"
date="20220415"
Copy link
Member

Choose a reason for hiding this comment

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

Let's put the latest articles at the top of this section. 😉
Could you reorder the articles?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, sure.
(Only videos in Korean were ordered differently 😆)

Copy link
Member Author

Choose a reason for hiding this comment

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

reorder

<Article
title="Armeria: 어디서나 잘 어울리는 마이크로서비스 프레임워크"
url="https://tv.naver.com/v/11267414"
author="이희승"
Copy link
Member

Choose a reason for hiding this comment

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

@ikhoon I think you forgot to thank @trustin.

Copy link
Member

@minwoox minwoox left a comment

Choose a reason for hiding this comment

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

Thanks! 😄

@ikhoon ikhoon merged commit 3da7bac into line:master Sep 7, 2022
@syleeeee syleeeee deleted the youtube-kr branch September 7, 2022 09:25
heowc pushed a commit to heowc/armeria that referenced this pull request Sep 24, 2022
We had Youtube live last April, and the site only showed the slides so far.
The LINE DevRel team posted the recorded videos, so I'd like to add them to the site as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants