Skip to content

feat: add custom api endpoint for Kaspa#1429

Open
anhductn2001 wants to merge 2 commits intomainfrom
ducnt/wrong_default_kaspa
Open

feat: add custom api endpoint for Kaspa#1429
anhductn2001 wants to merge 2 commits intomainfrom
ducnt/wrong_default_kaspa

Conversation

@anhductn2001
Copy link
Contributor

@anhductn2001 anhductn2001 commented Dec 10, 2025

PR Standards

Close #1426

Opening a pull request should be able to meet the following requirements

For Author:

  • Targeted PR against correct branch
  • Included the correct type prefix in the PR title
  • Linked to Github issue with discussion and accepted design
  • Targets only one github issue
  • Wrote unit and integration tests
  • All CI checks have passed
  • Added relevant godoc comments

For Reviewer:

  • Confirmed the correct type prefix in the PR title
  • Reviewers assigned
  • Confirmed all author checklist items have been addressed

After reviewer approval:

  • In case PR targets main branch, PR should be squashed and merged.
  • In case PR targets a release branch, PR should be rebased.

@anhductn2001 anhductn2001 requested a review from a team as a code owner December 10, 2025 10:11
artemijspavlovs
artemijspavlovs requested changes Dec 15, 2025
View reviewed changes
data_layer/kaspa/kaspa.go Outdated
kaspaConfig.ApiUrl = daData.ApiUrl
kaspaConfig.Root = root

useExistingApiEndpoint, _ := pterm.DefaultInteractiveConfirm.WithDefaultText(
Copy link
Collaborator

@artemijspavlovs artemijspavlovs Dec 15, 2025

Choose a reason for hiding this comment

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

overrideApiEndpoint would be more descriptive

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

Reviewers

@artemijspavlovs artemijspavlovs artemijspavlovs requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

wrong default kaspa da config

2 participants

@anhductn2001 @artemijspavlovs