[Wip] Youtube App #1086

JonasJesus42 · 2025-04-04T05:41:52Z

What is this Contribution About?

Please provide a brief description of the changes or enhancements you are proposing in this pull request.

Issue Link

Please link to the relevant issue that this pull request addresses:

Issue: #ISSUE_NUMBER

Loom Video

Record a quick screencast describing your changes to help the team understand and review your contribution. This will greatly assist in the review process.

Demonstration Link

Provide a link to a branch or environment where this pull request can be tested and seen in action.

…ovements

…d videos on YouTube

…uding actions for liking, pinning, and replying to comments, as well as live streaming management

…module files, including improvements to readability and code style consistency

…hentication and sections structure, removing unnecessary code and improving organization

github-actions · 2025-04-04T05:42:08Z

Tagging Options

Should a new tag be published when this PR is merged?

👍 for Patch 0.67.1 update
🎉 for Minor 0.68.0 update
🚀 for Major 1.0.0 update

…nd loaders, removing unnecessary code and improving project organization

matheusgr

Translate to English.

matheusgr · 2025-04-08T13:03:39Z

Youtube/README.md

+## Observações
+
+- Para modificar informações do canal, você deve ser o proprietário do canal ou ter permissões adequadas.
+- Para pinar comentários, você deve ser o proprietário do vídeo ou ter permissões adequadas.
+- Para acessar dados de analytics, você precisa ter o escopo `yt-analytics.readonly` autorizado.
+- Algumas operações exigem tokens de acesso com escopos específicos.
+- A API do YouTube tem restrições específicas sobre quais partes podem ser atualizadas em conjunto. Por exemplo, `snippet` e `brandingSettings` devem ser atualizados em chamadas separadas.
+- Operações como curtir comentários, definir status de moderação e outras ações sem conteúdo devem usar a abordagem de parâmetros na URL, não no corpo da requisição.


Put that above the resources.

It seems that most problems that uses may have are relate to this... so, keep it up on the doc.

matheusgr · 2025-04-08T13:03:52Z

Youtube/README.md

@@ -0,0 +1,631 @@
+# App do YouTube


English.... :) All the way.

matheusgr · 2025-04-08T13:07:07Z

shopify/utils/storefront/storefront.graphql.gen.ts

+export type GetShopInfoQueryVariables = Exact<{
+  identifiers: Array<HasMetafieldsIdentifier> | HasMetafieldsIdentifier;
+}>;
+
+
+export type GetShopInfoQuery = { shop: { name: string, description?: string | null, privacyPolicy?: { title: string, body: string } | null, refundPolicy?: { title: string, body: string } | null, shippingPolicy?: { title: string, body: string } | null, subscriptionPolicy?: { title: string, body: string } | null, termsOfService?: { title: string, body: string } | null, metafields: Array<{ description?: string | null, key: string, namespace: string, type: string, value: string, reference?: { image?: { url: any } | null } | {} | null, references?: { edges: Array<{ node: { image?: { url: any } | null } | {} }> } | null } | null> } };
+


Suggested change

export type GetShopInfoQueryVariables = Exact<{

identifiers: Array<HasMetafieldsIdentifier> | HasMetafieldsIdentifier;

}>;

export type GetShopInfoQuery = { shop: { name: string, description?: string | null, privacyPolicy?: { title: string, body: string } | null, refundPolicy?: { title: string, body: string } | null, shippingPolicy?: { title: string, body: string } | null, subscriptionPolicy?: { title: string, body: string } | null, termsOfService?: { title: string, body: string } | null, metafields: Array<{ description?: string | null, key: string, namespace: string, type: string, value: string, reference?: { image?: { url: any } | null } | {} | null, references?: { edges: Array<{ node: { image?: { url: any } | null } | {} }> } | null } | null> } };

should be here?

matheusgr · 2025-04-08T13:09:46Z

lives/types.ts

this seems like a youtube specific file. not something that is universal.

( https://schema.org/VideoObject is the most related schemaOrg that I could find).

Move to youtube app.

matheusgr · 2025-04-08T13:10:20Z

deco.ts

@@ -10,6 +10,7 @@ const compatibilityApps = [{

 const config = {
  apps: [
+    app("Youtube"),


avoid upper case on app names and directories

matheusgr · 2025-04-08T13:15:28Z

Youtube/loaders/videos/captionText.ts

+/**
+ * Parse SRT format captions
+ */
+function parseSrtCaption(


utility functions, I would keep apart from the loader.

matheusgr · 2025-04-08T13:16:35Z

Youtube/loaders/videoCategories.ts

+  try {
+    // Construir a URL para buscar as categorias de vídeos
+    const url =
+      `https://www.googleapis.com/youtube/v3/videoCategories?part=snippet&regionCode=${regionCode}`;


it seems that the google base API address should be something defined on the mod.ts, and reused here and on the other API URLs.

matheusgr · 2025-04-08T13:17:27Z

Youtube/loaders/livestreams/list.ts

+      if (response.status === 401) {
+        mensagem =
+          "Token de autenticação inválido ou expirado. Faça login novamente.";
+      } else if (response.status === 403) {
+        mensagem =
+          "Acesso negado. Verifique se você tem permissão para listar transmissões e o escopo 'youtube.livestream' está autorizado.";
+      } else if (response.status === 404) {
+        mensagem =
+          "Recurso não encontrado. Verifique se os IDs das transmissões estão corretos.";
+      } else if (response.status === 400) {
+        mensagem = "Solicitação inválida. Verifique os parâmetros enviados.";
+      } else if (response.status === 500) {
+        mensagem =
+          "Erro interno do servidor YouTube. Tente novamente mais tarde.";
+      } else if (response.status === 429) {
+        mensagem =
+          "Limite de requisições excedido. Aguarde um momento e tente novamente.";
+      }


this is something that you may need to do for every response.

matheusgr · 2025-04-08T13:20:51Z

Youtube/actions/comments/like.ts

+    return {
+      success: false,
+      message: `Erro ao avaliar comentário: ${error.message}`,
+    };


you should create a uniform API.

you can either... all return success with { data: any, error: Error }, which is a more common patter for APIs, or return data if successful, and throw an exception otherwise.

Usually I am more prone to the return data/exception for user facing APIs and data/error pattern for libraries (such as postgres).

This success/message seems confusing, since you are probably defining a UI message to be set by the API. And message means two different things here (error message or successful message if the success is true or false).

matheusgr · 2025-04-08T13:24:49Z

Youtube/actions/updateChannel.ts

+    return { success: false, message: "Autenticação necessária" };
+  }
+
+  try {


try catchs are too big on your functions. create a separate call for that logic, and wrap it inside here.

try { trueActionFn(props); } catch (err) { // do something }

matheusgr · 2025-04-08T13:34:14Z

Youtube/README.md

any reason why not to use the google-data-api?

…code, as well as update the import structure and improve code readability

…tor the access token management framework, including adding functions for cookie handling

… functionality in various YouTube actions and uploaders

…be API, improving type structure and code readability

…rs to English, improving code consistency and clarity

…o get the access token and channel data, and moving the default authentication URL to constants

…ode and adjusting the return structure of the authentication function

…g new fields and improving handling of YouTube API responses

…ing logic, including error handling and improved response structure

…ubtitle text loader and improving the response structure and configuration options

…Tube channels and videos, improving the efficiency and structure of responses

…regamento de canais e vídeos do YouTube, melhorando a eficiência e a estrutura das respostas

…ing YouTube video comments and categories, improving response structure and code efficiency

…ndling in YouTube API requests, improving response structure and code efficiency

…details interface, improving response structure and authentication logic

…tions, improving error handling, and refining response structure for better performance

…s on YouTube, adding new error interfaces and improving the structure of responses for greater efficiency

…rfaces. Standardize authentication via getAccessToken(req). Refactor and clean up YouTube integration

…ypes

…pe cleanup

…rs, and improve type safety

…tility

JonasJesus42 added 12 commits February 24, 2025 05:06

feat: init integration youtube

911e759

feat: init integration youtube

8c8c489

feat: Improve YouTube channel authentication and loading

ed1d6d0

feat: Improve YouTube channel authentication and loading

5955e24

feat: Add YouTube video upload and integration in section

5ec6bd0

feat: Add YouTube video refresh functionality and authentication impr…

374d20d

…ovements

merge

53465cd

feat: Add functionality to update YouTube video thumbnails

5a0aba4

feat: Refactor response types and update interfaces for thumbnails an…

df2ea2a

…d videos on YouTube

feat: Add new commenting and live streaming features to YouTube, incl…

253e02e

…uding actions for liking, pinning, and replying to comments, as well as live streaming management

refactor: Adjust code formatting and organization in several YouTube …

48a4332

…module files, including improvements to readability and code style consistency

feat: Add new action to delete videos on YouTube and refactor the aut…

4ccbfe5

…hentication and sections structure, removing unnecessary code and improving organization

refactor: Update data types to 'unknown' in several YouTube actions a…

d60ecc3

…nd loaders, removing unnecessary code and improving project organization

matheusgr changed the title ~~Wip Wip Wip~~ [Wip] Youtube-like API Apr 8, 2025

matheusgr changed the title ~~[Wip] Youtube-like API~~ [Wip] Youtube App Apr 8, 2025

matheusgr requested changes Apr 8, 2025

View reviewed changes

matheusgr approved these changes Apr 8, 2025

View reviewed changes

Youtube/README.md

Copy link

Contributor

matheusgr Apr 8, 2025

Choose a reason for hiding this comment

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

any reason why not to use the google-data-api?

JonasJesus42 added 12 commits April 16, 2025 04:30

refactor: Adjust authentication configuration and remove unnecessary …

1ff7f65

…code, as well as update the import structure and improve code readability

feat: Implement OAuth authentication client for YouTube API and refac…

b1d76f5

…tor the access token management framework, including adding functions for cookie handling

refactor: Update access token import to utilize new cookie management…

49092f0

… functionality in various YouTube actions and uploaders

refactor: Update client interfaces for OAuth authentication and YouTu…

d213a6f

…be API, improving type structure and code readability

refactor: Update descriptions and titles of YouTube actions and loade…

b755461

…rs to English, improving code consistency and clarity

refactoring: organizing folders and files and improving names

b055ecb

refactor: reorganize the authentication logic, extracting functions t…

db95820

…o get the access token and channel data, and moving the default authentication URL to constants

refactor: simplify the authentication logic by removing unnecessary c…

5045164

…ode and adjusting the return structure of the authentication function

refactor: Update the type structure and video loading logic, includin…

96a3567

…g new fields and improving handling of YouTube API responses

refactor: Remove unnecessary logs and update the channel's video load…

040075d

…ing logic, including error handling and improved response structure

refactor: reorganize YouTube's subtitle loading logic, removing the s…

9bb589e

…ubtitle text loader and improving the response structure and configuration options

refactor: add caching logic and configuration options for loading You…

046838a

…Tube channels and videos, improving the efficiency and structure of responses

JonasJesus42 added 14 commits April 17, 2025 08:40

refactor: adicionar lógica de cache e opções de configuração para car…

c0ec536

…regamento de canais e vídeos do YouTube, melhorando a eficiência e a estrutura das respostas

refactor: add configuration options and error handling logic for load…

1756b54

…ing YouTube video comments and categories, improving response structure and code efficiency

refactor: implement expired token checking logic and improve error ha…

93e9bfc

…ndling in YouTube API requests, improving response structure and code efficiency

refactor: add new fields and improve error handling in YouTube video …

4c5c23e

…details interface, improving response structure and authentication logic

refactor: enhance YouTube video search functionality by adding new op…

954407c

…tions, improving error handling, and refining response structure for better performance

refactor: improve the logic for deleting, updating, and loading video…

7acca1c

…s on YouTube, adding new error interfaces and improving the structure of responses for greater efficiency

Merge branch 'main' of github.com:deco-cx/apps into youtube

8a17f4a

fix: remove all usage of tokenYoutube from loaders, actions, and inte…

1ee2341

…rfaces. Standardize authentication via getAccessToken(req). Refactor and clean up YouTube integration

fix: default lowercase youtube in paths and restore type/interface names

9ce085b

refactor: completely remove tokenYoutube from actions, loaders, and t…

50f5208

…ypes

refactor(youtube): remove all usage of getAccessToken utility

37c6247

refactor(youtube): remove cookie-based token, unify auth, and lint/ty…

f7761fa

…pe cleanup

refactor(youtube): migrate all API calls to use client, fix type erro…

249e4f4

…rs, and improve type safety

feat(youtube): add support for OAuth token refresh and auto-refresh u…

620340b

…tility

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Wip] Youtube App #1086

[Wip] Youtube App #1086

Uh oh!

JonasJesus42 commented Apr 4, 2025

Uh oh!

github-actions bot commented Apr 4, 2025

Uh oh!

matheusgr left a comment

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

matheusgr Apr 8, 2025

Uh oh!

Uh oh!

[Wip] Youtube App #1086

Are you sure you want to change the base?

[Wip] Youtube App #1086

Uh oh!

Conversation

JonasJesus42 commented Apr 4, 2025

What is this Contribution About?

Issue Link

Loom Video

Demonstration Link

Uh oh!

github-actions bot commented Apr 4, 2025

Tagging Options

Uh oh!

matheusgr left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!