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: send tweet on new video post #328

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

falces
Copy link

@falces falces commented Oct 18, 2022

Hola! He añadido la funcionalidad de enviar un Tweet para promocionar un vídeo. Tengo la duda sobre una funcionalidad: a partir del id del vídeo, he creado un texto que incluye el nombre del curso con el que está relacionado, por lo que he tenido que ir al repositorio de cursos, pero hay dos que implementan la interface. Tenía las dudas:

  • Desde el dominio importar directamente el repositorio concreto (archivo o base de datos) a utilizar
  • Inyectar la interface CourseRepository en la capa aplicación y en services.yaml configurar el parámetro para decir a Symfony qué clase de las dos que instancian la interface debe utilizar (esta es la opción que he utilizado)

¡Muchas gracias!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant