feat(view patient): add 'edit' icon to Edit button

[MatthewDorner]

Fixes #1765

• Add 'edit' pencil icon to "Edit" button on View Patient component.

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/hospitalrun/hospitalrun-frontend/4uqes0dz3
✅ Preview: https://hospitalrun-frontend-git-fork-matthewdorner-add-pencil-icon.hospitalrun.now.sh

[jackcmeyer]

LGTM 🎉

[MatthewDorner]

These test failures are confusing to me. One is failing due to 'document update conflict' and another one is failing because the storybook (?) part of the test script failed. But I don't even see storybook in the other test runs that passed. Have we still been having test problems? I just looked at GitHub actions on master branch and it looks like most are passing...

edit: haha I guess they are all failing, must not have looked at it since yesterday... I will try to look into this

[matteovivona]

@MatthewDorner storybook check fails due to my error. I corrected github action conf file this morning

[MatthewDorner]

Yeah, I guessed the storybook thing was just added by mistake. Otherwise it looks like it's failing on Windows and the failure (on this and pretty much all the CI runs) is related to the tests that use PouchDB. Though I have seen those "document update conflict" errors a few times on my Linux environment, lately everything's been passing for me locally.

[matteovivona]

@HospitalRun/core-maintainers any of you guys got windows?

[MatthewDorner]

I can play around with it on a VM when I get some time. It's not just this PR, though, seems to have started failing this way on all the CI runs a few days ago...

[jackcmeyer]

@tehkapa I have a windows machine and I am unable to reproduce.

[vercel]

You are pushing commits at a very fast pace (across the whole organization).
Due to that, we cannot deploy the commit e786c74.

You can try again later or upgrade your plan.