Fix: Handle Principal IDs in the front-end

dfinity / linkedup

An open professional network.

https://sdk.dfinity.org

Apache License 2.0

154 stars 43 forks source link

Fix: Handle Principal IDs in the front-end #27

Closed ghost closed 4 years ago

ghost commented 4 years ago

Overview

Introduction of Principal IDs as the id type meant we couldn't use profile.id in the front-end in the same way. This PR introduces a naive application state for LinkedUp and manages actions that way.

Changes

Declare an application state to store search results and connections
Refer to search results / connections by their index instead of profile.id
Prettier

Note: Prettier wreaked havoc on formatting, so I'll explicitly call out changes as comments.

lsgunnlsgunn commented 4 years ago

Quick test--Search result is returned and I see the Connect button, but clicking it didn't seem to do anything (or it is taking a really long time).

Refreshing the browser after doing the above pulled in the connection. Yippee ....