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

Starknet Custom Release #13584

Merged
merged 6 commits into from
Jun 17, 2024
Merged

Starknet Custom Release #13584

merged 6 commits into from
Jun 17, 2024

Conversation

augustbleeds
Copy link
Contributor

@augustbleeds augustbleeds commented Jun 17, 2024 •
edited
Loading

Ticket

https://smartcontract-it.atlassian.net/browse/BCI-3447

Create the custom release for Starknet.

Features:

  • Enables nodes running Starknet to input a GasPriceSubunits pipeline and transmits gas price data on chain.
  • Updates FMS and operator-ui to accept the contract account address (transmitterId is the public key)

Brings in the following cherry-picks:
#13421
#13536
#13200

The following fixes have been made:

  • 97e8d48 (adds missing resolver that does nothing) . This is committed directly to this branch as fix

The last cherry-pick:

So... the first 2 are cherry-picks. The 3rd is a fix. And the 4th is a cherry-pick with module updates to only pull necessary changes
image

eutopian and others added 3 commits June 14, 2024 14:26
@eutopian @chris-de-leon-cll @augustbleeds
Add account address pub key to fms chain config for Starknet (#13421)
174befa 
* add account address pub key to fms chain config for starknet

* Update core/web/resolver/feeds_manager_chain_config.go

Co-authored-by: Chris De Leon <[email protected]>

---------

Co-authored-by: Chris De Leon <[email protected]>
@eutopian @augustbleeds
bump operator-ui to v0.8.0-7e6a14f (#13536)
b554e37
@eutopian @augustbleeds
fix standardcapabilitiesspec break gql updates for starknet release
97e8d48
@augustbleeds augustbleeds requested review from a team and RensR as code owners June 17, 2024 17:39
@augustbleeds augustbleeds force-pushed the augustus.starknet-patch branch from f7bafa9 to 043e476 Compare June 17, 2024 17:45
@augustbleeds augustbleeds changed the title Augustus.starknet patch Starknet Custom Release Jun 17, 2024
@augustbleeds @bolekk
BCI-1426: Add GasPriceSubunits Data Source Pipeline (#13200)
0115ac1 
* initial

* incorporate gasPriceSubunits data source

* update deps

* add in data source

* comment

* update plugin config

* add changeset

* update chainlink-common & chainlink-feeds

* lint fixes

* lint

---------

Co-authored-by: Bolek <[email protected]>
@augustbleeds augustbleeds force-pushed the augustus.starknet-patch branch from 043e476 to 0115ac1 Compare June 17, 2024 18:10
eutopian
eutopian previously approved these changes Jun 17, 2024
View reviewed changes
@augustbleeds augustbleeds dismissed stale reviews from chris-de-leon-cll and eutopian via 9d62574 June 17, 2024 20:43
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
81.0% Coverage on New Code
3.4% Duplication on New Code

See analysis details on SonarQube

@snehaagni snehaagni merged commit f5fab34 into release/2.12.1-starknet Jun 17, 2024
115 checks passed
@snehaagni snehaagni deleted the augustus.starknet-patch branch June 17, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris-de-leon-cll chris-de-leon-cll chris-de-leon-cll approved these changes

@eutopian eutopian eutopian approved these changes

@RensR RensR Awaiting requested review from RensR

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

Successfully merging this pull request may close these issues.
4 participants
@augustbleeds @eutopian @chris-de-leon-cll @snehaagni