Skip to content

Conversation

@dugajean
Copy link
Contributor

@dugajean dugajean commented Sep 9, 2020

No description provided.

Copy link
Contributor

@pereirinha pereirinha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dugajean

Can you please check the minor comments?

@pereirinha
Copy link
Contributor

@dugajean

Looks good to me and it's working fine.
Can you please fix the conflicts?

Copy link
Contributor

@pereirinha pereirinha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dugajean

The PR is approved, but we need to resolve the merge conflicts before.
Don't merge it once you resolve. I'll take care of that part.

# Conflicts:
#	cloudinary-image-management-and-manipulation-in-the-cloud-cdn/js/block-editor.js
#	cloudinary-image-management-and-manipulation-in-the-cloud-cdn/js/block-editor.js.map
#	cloudinary-image-management-and-manipulation-in-the-cloud-cdn/js/src/components/featured-image.js
@pereirinha pereirinha merged commit 15a665b into develop Sep 30, 2020
@pereirinha pereirinha deleted the fix/transformation-checkbox-unclarity branch October 7, 2020 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants