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

Feature/add reports and relays entities #31

Conversation

Alann27
Copy link
Contributor

@Alann27 Alann27 commented Nov 20, 2024

Summary

  • Added entities related to claims and proof
  • Added summarized data for each block
  • Added missing fields to blocks and transactions

Issue

The entities related to claims and proof were missing.

Type of change

Select one or more:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have commented my code
  • I have performed a self-review of my own code; both comments & source code
  • I create and reference any new tickets, if applicable
  • I have left TODOs throughout the codebase, if applicable

@jorgecuesta jorgecuesta added the enhancement New feature or request label Nov 20, 2024
@jorgecuesta jorgecuesta changed the title Feature/add reports and relays entities [NOT MERGE BEFORE #29] Feature/add reports and relays entities Nov 20, 2024
@jorgecuesta
Copy link
Collaborator

@Alann27 Please rebase this on top of #29 and change base branch to it too.

schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
src/mappings/poktroll/relays.ts Outdated Show resolved Hide resolved
src/mappings/poktroll/relays.ts Outdated Show resolved Hide resolved
src/mappings/poktroll/reports.ts Outdated Show resolved Hide resolved
src/mappings/poktroll/reports.ts Show resolved Hide resolved
src/mappings/utils.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@bryanchriswhite bryanchriswhite left a comment

Choose a reason for hiding this comment

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

This one is also quite large - please prefer many smaller PRs over one large one!

I only did a cursory review, mostly focusing the schema changes.

schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
@jorgecuesta jorgecuesta changed the title [NOT MERGE BEFORE #29] Feature/add reports and relays entities Feature/add reports and relays entities Nov 20, 2024
@Alann27 Alann27 force-pushed the feature/add-reports-and-relays-entities branch from 4b0767f to 8c7519a Compare November 20, 2024 23:26
@jorgecuesta jorgecuesta merged commit fdc9657 into pokt-network:main Nov 20, 2024
4 checks passed
@jorgecuesta jorgecuesta deleted the feature/add-reports-and-relays-entities branch November 20, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants