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

fix: head report chain_id #14212

Merged
merged 2 commits into from
Aug 23, 2024
Merged

fix: head report chain_id #14212

merged 2 commits into from
Aug 23, 2024

Conversation

bukata-sa
Copy link
Contributor

OTI head report request protobuf regenerate + set chain_id

@bukata-sa bukata-sa requested review from a team as code owners August 23, 2024 10:26
jmank88
jmank88 previously approved these changes Aug 23, 2024
View reviewed changes
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@bukata-sa bukata-sa requested a review from jmank88 August 23, 2024 16:45
@bukata-sa bukata-sa added this pull request to the merge queue Aug 23, 2024
@bukata-sa bukata-sa removed this pull request from the merge queue due to a manual request Aug 23, 2024
@bukata-sa bukata-sa added this pull request to the merge queue Aug 23, 2024
Merged via the queue into develop with commit 25d2961 Aug 23, 2024
134 of 135 checks passed
@bukata-sa bukata-sa deleted the fix/head_report_proto branch August 23, 2024 17:28
@github-actions github-actions bot mentioned this pull request Aug 23, 2024
Draft
asoliman92 pushed a commit that referenced this pull request Aug 28, 2024
@bukata-sa
cherry pick: non-zero default period (#14206) + head report chain_id (#…
74a6b00 
…14212) (#1355)

cherry pick 

621e875

25d2961
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmank88 jmank88 jmank88 approved these changes

@fouadkada fouadkada fouadkada approved these changes

@EasterTheBunny EasterTheBunny Awaiting requested review from EasterTheBunny

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bukata-sa @fouadkada @jmank88