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.

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

GDA: Dev/citations component override #450

Merged
merged 10 commits into from
Jul 24, 2024

Conversation

YextSLu
Copy link

@YextSLu YextSLu commented Jul 18, 2024

Changes:

  • Added CitationsContainer prop to <GenerativeDirectAnswer> to allow custom components to be passed in
  • Updated test cases and test-site with new component passed through

J=CLIP-1369
TEST=manual|auto

Ran npm run build, npm run test, and npm run wcag and did manual testing on test-site

YextSLu added 4 commits July 18, 2024 11:50
J=CLIP-1369
TEST=manual|auto
J=CLIP-1369
TEST=manual|auto

Ran `npm run test` and `npm run wcag` and did manual testing on test-site
J=CLIP-1369
TEST=manual|auto

Ran `npm run test` and `npm run wcag` and did manual testing on test-site
@YextSLu YextSLu requested a review from a team as a code owner July 18, 2024 18:38
Copy link
Contributor

Current unit coverage is 92.17687074829932%
Current visual coverage is 79.31671283471837%
Current combined coverage is 92.66634288489558%

@coveralls
Copy link

coveralls commented Jul 18, 2024

Coverage Status

coverage: 85.286% (+0.004%) from 85.282%
when pulling f603b45 on dev/citations-component-override
into 34f207c on develop.

J=CLIP-1369
TEST=manual|auto

Ran `npm run test` and `npm run wcag` and did manual testing on test-site
YextSLu and others added 2 commits July 24, 2024 11:01
J=CLIP-1369
TEST=manual|auto

Ran `npm run test` and `npm run wcag` and did manual testing on test-site
YextSLu and others added 2 commits July 24, 2024 11:55
J=CLIP-1369
TEST=manual|auto

Ran `npm run test` and `npm run wcag` and did manual testing on test-site
@YextSLu YextSLu merged commit 132f5df into develop Jul 24, 2024
21 of 22 checks passed
@yen-tt yen-tt deleted the dev/citations-component-override branch July 26, 2024 18:01
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.

4 participants