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

feat: As a user, I want the indexed objects to contain some "audit" data #568

Conversation

alainncls
Copy link
Collaborator

@alainncls alainncls commented Mar 16, 2024

What does this PR do?

  1. Adds audit data to the main entities managed by the subgraph
  2. Makes all IDs lowercase for coherence
  3. New Linea Goerli subgraph available here
  4. New Linea mainnet subgraph available here

Related ticket

Fixes #357 and #421

Type of change

  • Chore
  • Bug fix
  • New feature
  • Documentation update

Check list

  • Unit tests for any smart contract change
  • Contracts and functions are documented

@alainncls alainncls self-assigned this Mar 16, 2024
@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.63%. Comparing base (f2874e6) to head (ed5654a).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #568   +/-   ##
=======================================
  Coverage   93.63%   93.63%           
=======================================
  Files          20       20           
  Lines         330      330           
  Branches       76       76           
=======================================
  Hits          309      309           
  Misses         10       10           
  Partials       11       11           

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

@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch from 5218fcf to f001efc Compare March 16, 2024 22:52
@alainncls alainncls force-pushed the dev branch 4 times, most recently from 581c2b6 to f607329 Compare March 23, 2024 10:27
@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch from f001efc to d002ea4 Compare April 8, 2024 13:03
Copy link

netlify bot commented Apr 8, 2024

Deploy Preview for verax-tutorial ready!

Name Link
🔨 Latest commit ed5654a
🔍 Latest deploy log https://app.netlify.com/sites/verax-tutorial/deploys/6614360397d47b0007041f68
😎 Deploy Preview https://deploy-preview-568--verax-tutorial.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.

@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch 2 times, most recently from 997d409 to ce1f0e3 Compare April 8, 2024 16:26
@alainncls alainncls marked this pull request as ready for review April 8, 2024 16:26
@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch 2 times, most recently from 7302254 to ccb3643 Compare April 8, 2024 16:50
@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch from ccb3643 to cfc4584 Compare April 8, 2024 18:20
fix: All indexed objects should have lowercase IDs
@alainncls alainncls force-pushed the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch from cfc4584 to ed5654a Compare April 8, 2024 18:22
@alainncls alainncls merged commit bf677f4 into dev Apr 9, 2024
13 checks passed
@alainncls alainncls deleted the feat/as-a-user-i-want-the-indexed-objects-to-contain-some-audit-data branch April 9, 2024 08:33
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.

[US] As a user, I want the indexed objects to contain some "audit" data
3 participants