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

[Bug] NodeURL should not autocompleted "/api/v1/" #279

Closed
3 tasks
JelleMillenaar opened this issue May 27, 2021 · 1 comment · Fixed by #270
Closed
3 tasks

[Bug] NodeURL should not autocompleted "/api/v1/" #279

JelleMillenaar opened this issue May 27, 2021 · 1 comment · Fixed by #270
Assignees
Labels
Bug Something isn't working.

Comments

@JelleMillenaar
Copy link
Collaborator

Description

Describe the task, this may be scoped beyond a single PR.

We provide a node URL and the client autocompletes this with /api/v1/ to do the API calls, but that is not a guaranteed setup.

Motivation

Describe the motivation for the feature to be developed.

Chronicle and user set up nodes may not have /api/v1/ in their endpoints.

To-do list

Create a task-specific to-do list . Please link PRs that match the To-do list item behind the item after it has been submitted.

  • Remove /api/v1/ from the client
  • Edit default nodes and (WASM) examples to write out nodeURLs to include /api/v1
  • Publish NPM dev version for WASM
@JelleMillenaar JelleMillenaar added the Bug Something isn't working. label May 27, 2021
@JelleMillenaar JelleMillenaar added this to Backlog in Framework Developments via automation May 27, 2021
@l1h3r
Copy link
Contributor

l1h3r commented May 27, 2021

I think the issue is in iota.rs (we don't build the endpoints) and is a duplicate of iotaledger/iota.rs#527

@l1h3r l1h3r mentioned this issue Jun 10, 2021
5 tasks
Framework Developments automation moved this from Backlog to Part of next Release Jun 11, 2021
@JelleMillenaar JelleMillenaar removed this from Part of next Release in Framework Developments Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants