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

Fix/open graph - use gatsby head #2573

Merged
merged 43 commits into from
Apr 5, 2024

Conversation

clari182
Copy link
Collaborator

@clari182 clari182 commented Jan 19, 2024

@clari182 clari182 marked this pull request as draft January 19, 2024 13:16
Copy link

netlify bot commented Jan 19, 2024

Deploy Preview for staging-aiid ready!

Name Link
🔨 Latest commit 0918485
🔍 Latest deploy log https://app.netlify.com/sites/staging-aiid/deploys/65aaaab63fbd5700082f060c
😎 Deploy Preview https://deploy-preview-2573--staging-aiid.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov bot commented Jan 19, 2024

Codecov Report

Attention: Patch coverage is 98.95288% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 85.76%. Comparing base (c4a08f7) to head (1cd3f55).

Files Patch % Lines
site/gatsby-site/src/components/HeadContent.js 90.90% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           staging    #2573      +/-   ##
===========================================
- Coverage    86.18%   85.76%   -0.43%     
===========================================
  Files          229      248      +19     
  Lines         7516     8331     +815     
  Branches      2416     2624     +208     
===========================================
+ Hits          6478     7145     +667     
- Misses         958     1089     +131     
- Partials        80       97      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Jan 19, 2024

1 failed test on run #2029 ↗︎

1 471 244 0 Flakiness 0

Details:

Fix/open graph - use gatsby head
Project: aiid-site Commit: 54c16ca851
Status: Failed Duration: 09:35 💡
Started: Apr 3, 2024 2:23 PM Ended: Apr 3, 2024 3:33 PM
Failed  cypress/e2e/integration/dynamicCite.cy.js • 1 failed test • Cypress e2e tests

View Output

Test Artifacts
Dynamic Cite pages > Should load dynamic Incident data Test Replay

Review all test suite changes for PR #2573 ↗︎

Copy link

@clari182 clari182 marked this pull request as ready for review January 19, 2024 15:55
@clari182 clari182 requested review from cesarvarela and removed request for cesarvarela January 19, 2024 15:56
@clari182 clari182 marked this pull request as draft January 19, 2024 16:25
@clari182
Copy link
Collaborator Author

@cesarvarela please hold on review thanks!

@clari182
Copy link
Collaborator Author

clari182 commented Apr 4, 2024

@kepae I fixed a few more things and merged staging. We should be good now!

@kepae kepae merged commit bd21bd3 into responsible-ai-collaborative:staging Apr 5, 2024
8 of 10 checks passed
@clari182 clari182 deleted the fix/open-graph branch April 5, 2024 16:22
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