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

graph cron: use totalCutHeight from Graph by adding up all heights #2814

Merged
merged 2 commits into from
Jan 21, 2025
Merged

graph cron: use totalCutHeight from Graph by adding up all heights #2814

merged 2 commits into from
Jan 21, 2025

Conversation

alber70g
Copy link
Member

@alber70g alber70g commented Jan 21, 2025 •
edited
Loading

  • use totalCutHeight from Graph by adding up all heights.
  • Add lastBlockHeight to message.
  • Adds types

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jan 21, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: 81f2c9e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR 
Some errors occurred when validating the changesets config:
The package or glob expression "e2e-docs" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@kadena/docs" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

@vercel Vercel
Copy link

vercel bot commented Jan 21, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kode-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 11:35am
rwa-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 11:35am
tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 11:35am
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
dev-wallet ⬜️ Skipped (Inspect) Jan 21, 2025 11:35am
explorer ⬜️ Skipped (Inspect) Jan 21, 2025 11:35am
marmalade-marketplace ⬜️ Skipped (Inspect) Jan 21, 2025 11:35am
proof-of-us ⬜️ Skipped (Inspect) Jan 21, 2025 11:35am

@vercel vercel bot temporarily deployed to Preview – proof-of-us January 21, 2025 11:07 Inactive
@vercel vercel bot temporarily deployed to Preview – dev-wallet January 21, 2025 11:07 Inactive
@vercel vercel bot temporarily deployed to Preview – marmalade-marketplace January 21, 2025 11:07 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer January 21, 2025 11:07 Inactive
@alber70g alber70g changed the title chore(graphCron): use totalCutHeight from Graph by adding up all heights. Add lastBlockHeight to message. Adds types graph cron: use totalCutHeight from Graph by adding up all heights Jan 21, 2025
sstraatemans
sstraatemans reviewed Jan 21, 2025
View reviewed changes
packages/apps/tools/src/scripts/graphCron/index.ts Outdated
});
}
} catch (e) {
await sendErrorMessage({
title: `There was a general issue with the ${props.env} graph cron job`,
msg: ``,
msg: e,
Copy link
Contributor

Choose a reason for hiding this comment

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

are we sure this is a string? or an object? if object this will fail. maybe stringify to be sure?

@vercel vercel bot temporarily deployed to Preview – dev-wallet January 21, 2025 11:21 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer January 21, 2025 11:21 Inactive
@vercel vercel bot temporarily deployed to Preview – proof-of-us January 21, 2025 11:21 Inactive
@vercel vercel bot temporarily deployed to Preview – marmalade-marketplace January 21, 2025 11:21 Inactive
@alber70g alber70g force-pushed the ag/graph-cron/use-sum-of-heights branch from 5f1f0fc to 81f2c9e Compare January 21, 2025 11:25
@vercel vercel bot temporarily deployed to Preview – marmalade-marketplace January 21, 2025 11:25 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer January 21, 2025 11:25 Inactive
@vercel vercel bot temporarily deployed to Preview – proof-of-us January 21, 2025 11:25 Inactive
@vercel vercel bot temporarily deployed to Preview – dev-wallet January 21, 2025 11:25 Inactive
@alber70g alber70g merged commit 4dfacc5 into main Jan 21, 2025
18 checks passed
@alber70g alber70g deleted the ag/graph-cron/use-sum-of-heights branch January 21, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sstraatemans sstraatemans sstraatemans 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.
2 participants
@alber70g @sstraatemans