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

registry dapp: manage records #2323

Merged
merged 19 commits into from Oct 8, 2016
Merged

registry dapp: manage records #2323

merged 19 commits into from Oct 8, 2016

Conversation

derhuerst
Copy link
Contributor

@derhuerst derhuerst commented Sep 26, 2016

This PR addresses #2190.

I didn't test if a correct transaction is being sent due to ethcore/contracts#df32dce not having been deployed yet. I was lazy :P

@parity-cla-bot
Copy link

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

Many thanks,

Ethcore CLA Bot

@derhuerst derhuerst added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M5-ui labels Sep 26, 2016
@jacogr jacogr mentioned this pull request Sep 26, 2016
6 tasks
@jacogr
Copy link
Contributor

jacogr commented Sep 26, 2016

You can test with this address - 0x2b63aB9b01072328E45F6eC19f301FB4aAb56748

If you are happy (that is the latest contract), we can remove the hardcoded address and do it properly with a Parity PR.

@derhuerst
Copy link
Contributor Author

derhuerst commented Sep 27, 2016

Setting an A record seems to work, the transactions seems to be correct. 🙌

The name lookup in the registry dapp seems to have a bug however. I registered foobaz and set A to 0x1010101010101010101010101010101010101010. Later, looking A of foobaz up yields 0x1010101010101010000000000000000000000000. See #2354.

I also noticed the DataChanged event log is broken. Will investigate. The jsapi responds with no DataChanged events. DataChanged event works.

@derhuerst derhuerst added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Sep 27, 2016
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. and removed A0-pleasereview 🤓 Pull request needs code review. A8-looksgood 🦄 Pull request is reviewed well. labels Sep 28, 2016
* js: (228 commits)
  registration in place
  Backports to master (#2530)
  lookup hash
  ethcore_hashContent call
  single input for commit/filename
  basic githubhint layout
  Handle reorganizations in the state cache (#2490)
  terminate after 30 seconds (#2513)
  allow updates of the secure token
  Using pending block only if not old (#2514)
  Caching optimizations (#2505)
  rework connection display
  basic test for manual token
  Fixed overflow panic in handshake_panic (#2495)
  Trim password from file (#2503)
  Fixing RPC Filter conversion to EthFilter (#2500)
  init token updates take place
  initial token connection - WIP
  Fixing error message for transactions (#2496)
  Adjustable stack size for EVM (#2483)
  ...

# Conflicts:
#	js/src/dapps/registry/Application/application.js
#	js/src/dapps/registry/Container.js
#	js/src/dapps/registry/actions.js
#	js/src/dapps/registry/reducers.js
@jacogr jacogr removed the A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. label Oct 8, 2016
@jacogr
Copy link
Contributor

jacogr commented Oct 8, 2016

Applied all the needed fixes to get this back to operational. Including code cleanups, not using class functions like normal functions (there is a reason they are part of the class), fixing registering using sha3 values when it should just pass through, not using components from the main app (dapps are seperate), properly displaying the events using tables so the alignment is done properly, some of the ugly linting issues, etc.

@jacogr jacogr merged commit b10f164 into js Oct 8, 2016
@jacogr jacogr deleted the jr-registry-manage-records branch October 8, 2016 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants