Create a Provider class for the Cloudflare Stream API methods. Right now they are in the plugin class methods which is not an OOP approach.

CommentFileSizeAuthor
#5 3097317-cloudflare_stream_api_provider-5.patch25.3 KBtim-diels

Comments

sorabh.v6 created an issue. See original summary.

pjbaert’s picture

pjbaert
Primary language Dutch
commented
Assigned: sorabh.v6 » Unassigned

Unassigned since there was no activity

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Version: 8.x-1.x-dev » 8.x-2.x-dev

Lets do this in the 2.x version

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Assigned: Unassigned » tim-diels
tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Assigned: tim-diels » Unassigned
Status: Active » Needs review
StatusFileSize
new 3097317-cloudflare_stream_api_provider-5.patch25.3 KB

  • pjbaert committed 47e322e on 8.x-2.x authored by tim-diels 
    Issue #3097317 by tim-diels, pjbaert: Cloudflare Stream API Provider
pjbaert’s picture

pjbaert
Primary language Dutch
commented
Status: Needs review » Fixed

Just WOW !

Thanks for this huge rework. Pushed to 2.x branch!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.