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

Add BlockEnergyLimit Update #343

Merged
merged 5 commits into from
Sep 19, 2023
Merged

Add BlockEnergyLimit Update #343

merged 5 commits into from
Sep 19, 2023

Conversation

shjortConcordium
Copy link
Contributor

@shjortConcordium shjortConcordium commented Sep 14, 2023 •
edited
Loading

Purpose

Support the new BlockEnergyLimit Update.

Changes

  • Added the flow for the update.
  • Added a check for consensus update protocol.

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

@shjortConcordium shjortConcordium force-pushed the update-block-energy-limit branch from 5f251b4 to e540055 Compare September 14, 2023 16:04
Base automatically changed from gRPC2-migration to main September 19, 2023 10:52
orhoj
orhoj suggested changes Sep 19, 2023
View reviewed changes
app/utils/blockSummaryHelpers.ts Outdated Show resolved Hide resolved
@orhoj orhoj assigned shjortConcordium and unassigned orhoj Sep 19, 2023
@orhoj orhoj assigned shjortConcordium and unassigned orhoj Sep 19, 2023
@shjortConcordium shjortConcordium merged commit be0e5cf into main Sep 19, 2023
@shjortConcordium shjortConcordium deleted the update-block-energy-limit branch September 19, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@orhoj orhoj orhoj approved these changes

Assignees

@shjortConcordium shjortConcordium

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shjortConcordium @orhoj @soer8647