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

[DPA-1155]: feat(chainconfig): support aptos #90

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

graham-chainlink
Copy link
Collaborator

@graham-chainlink graham-chainlink commented Oct 11, 2024

Description

Allow APTOS to be selected in the chainconfig modal instead of just EVM.

The goal is being able to support APTOS in core node.

JIRA: https://smartcontract-it.atlassian.net/browse/DPA-1155

Pending this PR to be merged for the new network field.

Testing

Screenshot 2024-10-12 at 9 44 41 am

Checklist

If this PR creates changes to the operator-ui itself, rather than tests, pipeline changes, etc. Then please create a changeset so that a new release is created, and the changelog is updated. See: https://github.com/changesets/changesets/blob/main/docs/adding-a-changeset.md#what-is-a-changeset

  • This PR has an accompanying changeset if needed.

@graham-chainlink graham-chainlink force-pushed the ggoh/DPA-1155/support-aptos-chain-config branch 5 times, most recently from e034f3d to a1205bd Compare October 14, 2024 23:35
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 53.84615% with 18 lines in your changes missing coverage. Please review.

Project coverage is 87.68%. Comparing base (b940c8d) to head (a1205bd).

Files with missing lines Patch % Lines
.../screens/FeedsManager/EditSupportedChainDialog.tsx 22.22% 7 Missing ⚠️
...c/screens/FeedsManager/NewSupportedChainDialog.tsx 22.22% 7 Missing ⚠️
src/components/Form/ChainConfigurationForm.tsx 73.33% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
+ Coverage   87.58%   87.68%   +0.10%     
==========================================
  Files         247      249       +2     
  Lines        4503     4524      +21     
  Branches      781      735      -46     
==========================================
+ Hits         3944     3967      +23     
- Misses        537      555      +18     
+ Partials       22        2      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@graham-chainlink graham-chainlink marked this pull request as ready for review October 14, 2024 23:46
@graham-chainlink
Copy link
Collaborator Author

graham-chainlink commented Oct 14, 2024

Unfortunately with the code coverage complains, just that the the current components are lacking unit tests, it will take some time to add all those missing unit tests, so i only added some for now.

@graham-chainlink graham-chainlink force-pushed the ggoh/DPA-1155/support-aptos-chain-config branch from a1205bd to 9db2c15 Compare October 15, 2024 00:38
Allow APTOS to be selected in the chainconfig modal instead of just EVM

JIRA: https://smartcontract-it.atlassian.net/browse/DPA-1155
@graham-chainlink graham-chainlink force-pushed the ggoh/DPA-1155/support-aptos-chain-config branch 2 times, most recently from e318285 to e623f73 Compare October 15, 2024 05:04
@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
44.2% Coverage on New Code (required ≥ 75%)
3.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube

@graham-chainlink graham-chainlink merged commit 352091e into main Oct 15, 2024
12 of 13 checks passed
@graham-chainlink graham-chainlink deleted the ggoh/DPA-1155/support-aptos-chain-config branch October 15, 2024 06:17
graham-chainlink added a commit to smartcontractkit/chainlink that referenced this pull request Oct 15, 2024
Pulling in the latest change to support APTOS in chain config UI

Reference: smartcontractkit/operator-ui#90

JIRA: https://smartcontract-it.atlassian.net/browse/DPA-1155
github-merge-queue bot pushed a commit to smartcontractkit/chainlink that referenced this pull request Oct 15, 2024
Pulling in the latest change to support APTOS in chain config UI

Reference: smartcontractkit/operator-ui#90

JIRA: https://smartcontract-it.atlassian.net/browse/DPA-1155
cedric-cordenier pushed a commit to smartcontractkit/chainlink that referenced this pull request Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants