Sir i am an outreachy applicant and this issue is not for newcomers but I would like to work on it if you allow.

not getting confirmation email

Hey @jflory7 , I would like to work in this issue. Could you please assign it to me?

@alishapapun Sure. Let's discuss the implementation first before jumping in. Do you have some thoughts on how to integrate video messages into the app?

I remember past discussions with @jonatoni and @bee2502 where we did not want to host the video files on the server. If I remember correctly, we want to add support for embedded videos, e.g. a YouTube, Twitter, and/or Vimeo videos. Someone could add a link to a video message and then it would be embedded in the Happiness Packet email. Or this is what I am picturing for this ticket.

@jonatoni @alishapapun Do you have some thoughts on how this should be implemented?

I think the idea of embedded videos will work fine @jflory7 . I need bit clarity of where it could possibly be present in the website and what exactly we the videos are concerned to!

I think the idea of embedded videos will work fine @jflory7 . I need bit clarity of where it could possibly be present in the website and what exactly we the videos are concerned to!

@jonatoni @bt0dotninja @anxh3l0 Do you any of you have thoughts on how this should be implemented? I have a hard time picturing how this is used. I am having second thoughts if this is an important feature (or if it is better to hold off until an intern starts), but I'm curious for what you all think.

@jonatoni, Iam an Outreachy applicant and I would like to work on this task, if you can please assign it to me. Thanks

Hii @jflory7 @jonatoni :)

I remember past discussions with @jonatoni and @bee2502 where we did not want to host the video files on the server. If I remember correctly, we want to add support for embedded videos, e.g. a YouTube, Twitter, and/or Vimeo videos. Someone could add a link to a video message and then it would be embedded in the Happiness Packet email.

I did a little research and agree with this. We can have a function in our message model to identify the links the users include in their messages. Then, we can embed the video back in the template using an iframe and the list of URLs the user had in the message and include it in the packet mail sent to the receiver. What do you think?

I think this is an interesting issue and would like to work on it. :smile:

@sanjana11147 This approach sounds good to me. :thumbsup: I think this plan is sufficient to start work on this ticket. I'll assign this one to you.

Let us know if we can do anything to help! :raised_hands:

@jflory7 how about if we could add a filefield in the model message where we we can able to give the user both sending a video and a message?.

@jflory7 i kindly wish to know the status of this issue

@jflory7 i kindly wish to know the status of this issue

@feisal This ticket is currently assigned to @sanjana11147. Please choose another task to work on.

@sanjana11147 @jflory7 As we are already adding a WYSIWYG Editor, we could handle embedding the video in the editor itself. This would simplify the job at the back-end without adding much complexity and all the content related part of mails can be handled at one place.
A number of Django WYSIWYG editors are available that provide support for video embedding. Froala is one such editor that has a django specific framework support called django-froala-editor.
@sanjana11147 @jflory7 I'd love to hear your thoughts on this.

@sanjana11147 @jflory7 I'd love to hear your thoughts on this.

Nice, I haven't heard of Froala before. I prefer implementing this in the front-end too and have it already be embedded in the email. It would also include emoji input to close #30. I think it makes sense to mark this as depending on #121.

@sanjana11147, what do you think? Would you want to take on #121 as a larger task to improve writing new Happiness Packet messages?

Hi, after reviewing this ticket, I decided to close this ticket as out of scope. In #121, we discovered we are unable to use the django-froala-editor. It may be possible for a different editor to handle video messages, but support for embedded video messages is not a deciding factor for what WYSIWYG editor we end up using.

Editors aside, the scope of work outlined in the requirements list makes me think we will have our hands full with other more important work first.