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.

Sign up for GitHub

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

Jump to bottom

feat(components/modals): confirm test harness #510

Merged
merged 10 commits into from
Sep 16, 2022
Merged

feat(components/modals): confirm test harness #510

merged 10 commits into from
Sep 16, 2022

Conversation

Blackbaud-ErikaMcVey
Copy link
Contributor

No description provided.

@nx-cloud
Copy link

nx-cloud bot commented Sep 12, 2022 •
edited
Loading

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 9bddd98. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

✅ Successfully ran 18 targets

Sent with 💌 from NxCloud.

@blackbaud-sky-build-user
Copy link
Collaborator

blackbaud-sky-build-user commented Sep 12, 2022 •
edited
Loading

Storybook preview

Component Storybooks:

Apps:

@Blackbaud-ErikaMcVey Blackbaud-ErikaMcVey marked this pull request as ready for review September 13, 2022 13:25
@Blackbaud-ErikaMcVey Blackbaud-ErikaMcVey changed the title confirm test harness feat(components/modals): confirm test harness Sep 13, 2022
Blackbaud-SteveBrush
Blackbaud-SteveBrush reviewed Sep 16, 2022
View reviewed changes
libs/components/modals/testing/src/public-api.ts Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-button-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-button-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-button-harness-filters.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-button-harness.ts Outdated Show resolved Hide resolved
libs/components/modals/testing/src/confirm/confirm-harness.ts Outdated Show resolved Hide resolved
@Blackbaud-ErikaMcVey Blackbaud-ErikaMcVey merged commit 3b373e9 into main Sep 16, 2022
@Blackbaud-ErikaMcVey Blackbaud-ErikaMcVey deleted the confirm-test-harness branch September 16, 2022 18:29
@blackbaud-sky-build-user blackbaud-sky-build-user mentioned this pull request Sep 16, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Blackbaud-PaulCrowder Blackbaud-PaulCrowder Blackbaud-PaulCrowder left review comments

@Blackbaud-CoreyArcher Blackbaud-CoreyArcher Blackbaud-CoreyArcher approved these changes

@Blackbaud-SteveBrush Blackbaud-SteveBrush Blackbaud-SteveBrush approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Blackbaud-ErikaMcVey @blackbaud-sky-build-user @Blackbaud-PaulCrowder @Blackbaud-SteveBrush @Blackbaud-CoreyArcher