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 videos from the wdio-video-reporter to the report output #162

Open
alexwalker1993 opened this issue May 25, 2023 · 12 comments
Open

Add videos from the wdio-video-reporter to the report output #162

alexwalker1993 opened this issue May 25, 2023 · 12 comments
Labels
help wanted Extra attention is needed

Comments

@alexwalker1993
Copy link

Is your feature request related to a problem? Please describe.
I want to add the videos from the wdio-video-reporter to the report but it doesn't seem to be possible

Describe the solution you'd like
The report to have the videos attached to the relevant tests

@christian-bromann
Copy link
Contributor

Thanks for reporting!

Any contributions that implements this feature are highly appreciated. Don't expect this to be picked up by active contributors as they have their own priorities. If you depend on this bug to be fixed, your contribution is required. Please take a look into our contribution guidelines or join our Discord development server and let us know if you have any questions. Cheers!

@christian-bromann christian-bromann added the help wanted Extra attention is needed label May 25, 2023
@mikezaschka
Copy link

I do have the same requirement and will potentially look into this in the next couple of weeks.

@christian-bromann
Copy link
Contributor

@mikezaschka any updates? Happy to help if anything is unclear.

@isopropilick
Copy link

I think it's a single weekend issue maybe? Can I take it?

@christian-bromann
Copy link
Contributor

@isopropilick that would be awesome, I think changes need to be made in the video reporter. I just rewrote the reporter and am about to release a new major version soon.

@isopropilick
Copy link

isopropilick commented Jan 3, 2024

@christian-bromann No problem! if your dev branch is public I would love to take a look and then return to this issue when the realase is made in order to use a tagged version of the video reporter to tackle this one.

@isopropilick
Copy link

Maybe a little JSON-based magic for metadata --- blob / arraybuffer for the transfer and/or store of the video --- and add the default base64 thinggy for images? IDK seems fun

@christian-bromann
Copy link
Contributor

It is, go for it!

@christian-bromann
Copy link
Contributor

No problem! if your dev branch is public I would love to take a look and then return to this issue when the realase is made

Actually I plan to do the release now.

@isopropilick
Copy link

@christian-bromann sweet I will look in to this this week!

@AlfredoMoretta
Copy link

hi @isopropilick any news about this topic? thank you in advance for the answer

@christian-bromann
Copy link
Contributor

@AlfredoMoretta I think this bug is still up for grabs, any contributions would be much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants