search

status-im / specs

Specifications for Status clients.
https://specs.status.im/
MIT License
14 stars 14 forks source link

Modify trust establishment based on Corey's draft #25

Closed oskarth closed 5 years ago

oskarth commented 5 years ago

https://notes.status.im/DxjyZ4RUTbm-GeUD9yQhCw?view

@corpetty I'm cleaning up the specs repo, and I took the liberty to import your doc here. If you prefer to do it yourself, feel free to open a new PR and close this one.

There's already an existing rough draft, and we probably want to decide what to do with the existing docs. When in doubt I'd err on us removing it and reducing scope for these specs.

oskarth commented 5 years ago

I think it'd be good if we could merge a rough draft of this and then iterate with specific GH issues. This way we have something done and avoid getting in situation like before, where we had a few PRs open for months due to big initial scope.

Lmk what you think @corpetty

Also cc @decanus

oskarth commented 5 years ago

@corpetty do you want to merge this or add commits on top to address Adam's review?

oskarth commented 5 years ago

Ping @corpetty

corpetty commented 5 years ago

I will add commits on top of this this week.

oskarth commented 5 years ago

This PR has been open for way too long, I'm going to merge this as partial progress and we can clean it up later.

@corpetty please look at the PR reviews here and address as you think is appropriate, i.e. fixup PRs/issues