Skip to content
This repository has been archived by the owner on Jul 9, 2021. It is now read-only.

Coordinator Registry #1675

Merged
merged 1 commit into from
Mar 7, 2019
Merged

Coordinator Registry #1675

merged 1 commit into from
Mar 7, 2019

Conversation

hysz
Copy link
Contributor

@hysz hysz commented Mar 6, 2019

Description

A basic Coordinator registry contract that allows a Coordinator operator to record the endpoint of their Coordinator, to be queried by the public.

Testing instructions

Types of changes

Checklist:

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons.

@hysz hysz force-pushed the feature/contracts/TECRegistry-2 branch from 47c8e04 to 6675efc Compare March 6, 2019 16:18
@hysz hysz requested review from abandeali1 and fabioberger March 6, 2019 16:24
@hysz hysz force-pushed the feature/contracts/TECRegistry-2 branch 3 times, most recently from 4f8d807 to 382e046 Compare March 6, 2019 23:59
@hysz hysz changed the title TEC Registry Coordinator Registry Mar 7, 2019
@coveralls
Copy link

coveralls commented Mar 7, 2019

Coverage Status

Coverage remained the same at 85.142% when pulling 93568f6 on feature/contracts/TECRegistry-2 into babe013 on development.

Copy link
Member

@abandeali1 abandeali1 left a comment

Choose a reason for hiding this comment

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

Small styling nit, then LGTM.


/// @dev Gets the endpoint for a Coordinator.
/// @param coordinatorOperator operator of the Coordinator endpoint.
function getCoordinatorEndpoint(address coordinatorOperator) external view returns (string memory coordinatorEndpoint) {
Copy link
Member

Choose a reason for hiding this comment

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

Can we put the modifiers on separate lines?


/// @dev Gets the endpoint for a Coordinator.
/// @param coordinatorOperator operator of the Coordinator endpoint.
function getCoordinatorEndpoint(address coordinatorOperator) external view returns (string memory coordinatorEndpoint);
Copy link
Member

Choose a reason for hiding this comment

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

Same as above.

@hysz hysz force-pushed the feature/contracts/TECRegistry-2 branch 2 times, most recently from e3f01f0 to ac4ad09 Compare March 7, 2019 19:13
"pr": 1675
}
]
},
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like we never published 0.0.1, let's roll these into one CHANGELOG entry.

@hysz hysz force-pushed the feature/contracts/TECRegistry-2 branch from ac4ad09 to bd86b59 Compare March 7, 2019 19:21
@hysz hysz force-pushed the feature/contracts/TECRegistry-2 branch from bd86b59 to 93568f6 Compare March 7, 2019 19:32
@hysz hysz merged commit 6bd1a5a into development Mar 7, 2019
@hysz hysz deleted the feature/contracts/TECRegistry-2 branch March 16, 2019 04:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants