-
Notifications
You must be signed in to change notification settings - Fork 622
Open
Description
While testing the contacts list demo, I found several issues. Listing them down and will raise a PR fixing all of them at once.
- Incorrect URL for the server (should be 10003 instead of 10002)
-
The "Message" button doesn't work for sending a message to the contact. Results in a blank screen
-
The contact's links (based on JSON data) are not "linkified". I.e. we can't click and open them
-
Usage of icons are broken (e.g. HTML ends up with
locationOninstead oflocation-on)
Also, making the prompt a bit dynamic to include correct icons of URL, location, calendar, etc
- This is more of an enhancement, but following and messaging a contact should result in a card/UI which shows the name of the person followed, and perhaps a back button. I've fixed that locally. Will include in the PR
--
I will raise a PR for all of these.
Metadata
Metadata
Assignees
Labels
No labels