Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

move utility functions to api.util #2105

Merged
merged 10 commits into from
Sep 19, 2016
Merged

move utility functions to api.util #2105

merged 10 commits into from
Sep 19, 2016

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Sep 16, 2016

The original format endpoint just exploded, and it started to not make sense from a API end-user perspective since everything in there was not formatting related and to make matters worse we started confusing the internal libraries with the external interfaces.

(Any new features added under api/format that is WIP, needs to be moved to this endpoint as well. For current code, a s/api.format/api.util/g will do the trick)

@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. M-UI labels Sep 16, 2016
@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 17, 2016
@jacogr jacogr added M6-rpcapi 📣 RPC API. and removed M-UI labels Sep 18, 2016
@jacogr jacogr changed the title jsapi - move utility functions to api.util move utility functions to api.util Sep 18, 2016
* js: (59 commits)
  use standard modal title functionality
  fix modal title size where no steps are available
  get adding addresses working again (context migration)
  Bump clippy & Fix warnings (#2109)
  Added docs
  fix merge conflict
  check for existence of deprecated ethash file before attempting delete (#2103)
  fix grumble
  check response.ok, throw Error
  array for classlist
  .map -> .forEach
  fix wording
  rename fundAccount to shapeshift
  use shapeshift ison & name
  fix typo
  use only one image for tokens, scale it (maps what is available when pulled form IMG meta in TokenReg)
  Import summary
  unknown image when none found for a token
  don't show tokens.value === 0 in transfer
  cater for the cases where the token doesn't have an image
  ...

# Conflicts:
#	js/src/modals/Transfer/Details/details.js
# Conflicts:
#	js/src/dapps/registry/Lookup/actions.js
#	js/src/dapps/registry/Status/status.js
@derhuerst derhuerst merged commit 61f455d into js Sep 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M6-rpcapi 📣 RPC API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants