Skip to content

[SDK-3090] Remote screenshotter service support #1

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

Merged
merged 11 commits into from
Oct 28, 2022
Merged

[SDK-3090] Remote screenshotter service support #1

merged 11 commits into from
Oct 28, 2022

Conversation

froodian
Copy link
Collaborator

This is an implementation of a command similar to the matchRemoteImageSnapshot command from IAM-76 / https://github.com/Appboy/platform/pull/46841 - rewritten on a fork of a more actively maintained, ts-based, image comparison cypress plugin (with a comparison/baseline-updating UI!). It's working for me locally integrated into platform. I'm not able to commit my platform side yet due to some tsconfig issues that I hope running the CI workflow (which is in github) on this repo will allow me to repair.

Copy link

@cellog cellog left a comment

Choose a reason for hiding this comment

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

nice!

@froodian
Copy link
Collaborator Author

froodian commented Oct 27, 2022

look at this, it's even mad at me for hurting the test coverage! I love this repo!

oh, no, it just can't run coverage because it doesn't have the secrets the base repo had

@Zaxcoding
Copy link

Looks like it got ported over cleanly to me

@froodian
Copy link
Collaborator Author

i think i have to merge it for github to pick up the workflow revision, all but the step that requires codeclimate are green

@froodian froodian merged commit 9afa0ee into main Oct 28, 2022
@froodian froodian deleted the SDK-3090 branch October 28, 2022 17:46
@github-actions
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants