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

US04 #2

Merged
merged 14 commits into from
Dec 12, 2024
Merged

US04 #2

merged 14 commits into from
Dec 12, 2024

Conversation

wChrstphr
Copy link

@wChrstphr wChrstphr commented Dec 8, 2024

Descrição

Fizemos a implementação das funções de favoritar e assistir mais tarde à prévia do vídeo. Assim como o menu drop-down com as mesmas funções para dispositivos mobile.

Porque este Pull Request é necessário?

Permite que o usuário favorite ou salve o vídeo pela prévia no Desktop e pelo Mobile .

Critérios de Aceitação

Checklist

  • Teste passaram
  • Build passaram
  • Sem conflitos com a Branch Main ou Development.

Resolve #1 .

@ErickMVdO ErickMVdO merged commit efa18a5 into develop Dec 12, 2024
1 check passed
Comment on lines 80 to 91
findVideoById(): void {
// Aqui você deve definir como pegar o idVideo, talvez com route ou outro método
// Como você já tem o idVideo, só faz a requisição
this.videoService.findVideoById(this.idVideo).subscribe({
next: (data) => {
// Aqui você pode armazenar o vídeo retornado, como já está no código original
},
error: (err) => {
console.error('Error fetching video details', err);
},
});
}

Choose a reason for hiding this comment

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

Um dos motivos para os testes estarem quebrando era que esta função não estava sendo usada para nada, apenas fazia a chamada para o service findVideoById. Portanto, removi a função do código.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Feito
Development

Successfully merging this pull request may close these issues.

US04 - Adicionar vídeo ao assistir mais tarde ou favoritos pela prévia do vídeo
6 participants