Problem/Motivation

The current version of the Cloudflare Stream Sync module is working & stable. Since this functionality is a use-full extension for the current version of this module, I suggest we integrate this.

CommentFileSizeAuthor
#4 cf-sync-overview-links-after.png15.1 KBtim-diels
#4 cf-sync-overview-after.png27.91 KBtim-diels
#4 cf-sync-overview-links-before.png9.28 KBtim-diels
#3 interdiff_2_3.txt2.82 KBtim-diels
#3 3240876-lets_integrate_the_cloudflare_stream_sync_module-3.patch17.97 KBtim-diels
#2 3240876-lets_integrate_the_cloudflare_stream_sync_module-2.patch16.38 KBpjbaert

Comments

pjbaert created an issue. See original summary.

pjbaert’s picture

pjbaert
Primary language Dutch
commented
Assigned: pjbaert » Unassigned
Status: Active » Needs review
StatusFileSize
new 3240876-lets_integrate_the_cloudflare_stream_sync_module-2.patch16.38 KB

This patch integrates the latest version of the Cloudflare stream sync module.
Please review.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
StatusFileSize
new 3240876-lets_integrate_the_cloudflare_stream_sync_module-3.patch17.97 KB
new interdiff_2_3.txt2.82 KB

Small adjustments to the placement of the links.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
StatusFileSize
new cf-sync-overview-links-before.png9.28 KB
new cf-sync-overview-after.png27.91 KB
new cf-sync-overview-links-after.png15.1 KB
tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented

Functionality is tested and working, didn't touch that. So should be ready to go in.

  • pjbaert committed 65ab832 on 8.x-2.x 
    Issue #3240876 by tim-diels, pjbaert: Let's integrate the Cloudflare...
pjbaert’s picture

pjbaert
Primary language Dutch
commented
Status: Needs review » Fixed

Thank you for your review & suggested updates. This was pushed to the dev branch.

Status: Fixed » Closed (fixed)

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