Skip to content

As a content editor, I want to be able to add videos from Brightcove. #78

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

Closed
3 tasks done
fureigh opened this issue Dec 8, 2017 · 13 comments
Closed
3 tasks done
Assignees

Comments

@fureigh
Copy link
Contributor

fureigh commented Dec 8, 2017

User story

As a content editor, I want to be able to add videos from Brightcove so that I can make good use of existing workflows and resources.

From #70.

Acceptance criteria

  • [ ]
  • [ ]
  • [ ]

Definition of done

  • [ ]
  • [ ]
  • [ ]

Outstanding questions

  • For this phase, do users need to be able to upload videos to Brightcove as well, or just add videos from Brightcove?
  • Which users should be able to do this? All content editors and admins?
  • @fureigh may need to request credentials in order to complete integration.
@fureigh
Copy link
Contributor Author

fureigh commented Dec 8, 2017

Answers from this afternoon's conversation:

  • Users just need to be able to add videos from Brightcove, not upload videos to Brightcove via this site.
  • Example in lieu of credentials:
<iframe src="//players.brightcove.net/679256133001/default_default/index.html?videoId=4805848656001" 
allowfullscreen 
webkitallowfullscreen 
mozallowfullscreen 
width="428.57" 
height="150"></iframe>

@RPentola
Copy link
Collaborator

pasting in my email comment from this morning. This is an area where we should definitely do user testing.

Many of the webmasters outside of OLPA are not code-first users, so we should explore a solution that allows a drag/drop approach to adding videos. (Rather than pasting and customizing the code in the page.)

I see that Drupal has integration with Brightcove through the media module. (https://www.drupal.org/project/media). Would this module provide a more content-focused user experience (as opposed to code-focused)? And is this something you are working on for the prototype?

@fureigh
Copy link
Contributor Author

fureigh commented Dec 12, 2017

@RPentola Yep, we'll definitely want to test this! I've started configuring the Brightcove module for the prototype, since you mentioned Brightcove integration in #70.

I want to be sure I understand: when you say drag-and-drop, do you mean dragging from Brightcove (in a browser window) to the Drupal site (in another browser window or tab)?

@RPentola
Copy link
Collaborator

drag & drop or WYSIWYG. The concept would be that a user would see a list of videos and be able to pick which one to use. Then the video would display in the page.

Video display: maybe not in-line with the text, but in a designated spot? I'm not sure how the module works, so I'm flying blind on this one.

@fureigh
Copy link
Contributor Author

fureigh commented Dec 15, 2017

Hmm — dragging and dropping a file that exists on one's own computer is supported, but dragging and dropping videos already uploaded to Brightcove would require significant customization and/or creating a new module. Let's circle back to this once we've given the out-of-the-box implementation a spin.

In the meantime, I'll need the following information to be able to finish configuring the out-of-the-box implementation:

brightcove-config

Who would have that information? (I strongly recommend sharing the answers via a tool like fugacio.us or over a phone call, as email isn't secure.)

@RPentola
Copy link
Collaborator

@fureigh Yes--let's go with the regular OOTB version to start.

I think Ellen Weir would have those answers about Brightcove. @EYWeir (Is this your notation, Ellen?)

@fureigh
Copy link
Contributor Author

fureigh commented Dec 20, 2017

Phone call scheduled for tomorrow morning. Thanks!

@RPentola
Copy link
Collaborator

@fureigh what time is the call? Could I participate, too?

@fureigh
Copy link
Contributor Author

fureigh commented Dec 21, 2017

@RPentola I started my day early to have the call, so am just seeing this now — pardon.

The purpose of the call was to confirm 1) the list of needed credentials from the screenshot above and 2) how David will send them. Both of those are on track. If you had additional aims, let me know and we can figure out how best to achieve them. Thanks!

fureigh referenced this issue Jan 4, 2018
fureigh referenced this issue Jan 4, 2018
Note that this composer.lock file reflects many changes. Once the
remaining deployment issues have been sorted out, come back to
this to verify that the automatic changes are all correct and
necessary.

https://github.com/18F/nsf/issues/78
@fureigh
Copy link
Contributor Author

fureigh commented Feb 8, 2018

This one has turned out to be a surprisingly heavy lift, but it's now ready for a look on dev-nsf (not yet on test-nsf). You should see a Brightcove icon next to the image icon in the WYSIWYG editor.

@fureigh
Copy link
Contributor Author

fureigh commented Feb 8, 2018

@RPentola says 👍 (Slack)

@fureigh fureigh closed this as completed Feb 8, 2018
@fureigh
Copy link
Contributor Author

fureigh commented Feb 8, 2018

(Reopening 'til it's up on live)

@fureigh fureigh reopened this Feb 8, 2018
@fureigh
Copy link
Contributor Author

fureigh commented Feb 12, 2018

Confirming: this is up on live and the latest videos have been imported.

@fureigh fureigh closed this as completed Feb 12, 2018
fureigh referenced this issue Mar 8, 2018
Follow-up on cd8d1dd52553e16847b505a168fc7a8b49f7e605, which was
already pushed.

https://github.com/18F/nsf/issues/78
fureigh referenced this issue Mar 8, 2018
This is a precursor to making it possible to embed Brightcove
videos via the WYSIWYG.

https://github.com/18F/nsf/issues/78
fureigh referenced this issue Mar 8, 2018
Toward being able to embed Brightcove videos via the WYSIWYG
editor in a user-friendly way.

https://github.com/18F/nsf/issues/78
fureigh referenced this issue Mar 8, 2018
Toward being able to embed Brightcove videos via the WYSIWYG
editor in a user-friendly way.

https://github.com/18F/nsf/issues/78
fureigh referenced this issue Mar 8, 2018
...and the Entity module, brought along by `composer require`.

https://github.com/18F/nsf/issues/78
fureigh added a commit that referenced this issue May 4, 2018
This was added in #78 as part of Brightcove integration but
appears to have fallen out of composer.json somewhere along the
way.

For further implementation context, see
https://pronovix.com/blog/brightcove-video-connect-drupal-8-part-2-installation-configuration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants