Closed iamsolankiamit closed 4 years ago
Looks good to me too @chrisys . As far as changing the account goes, we run into issue #49
this will addressed separately, as it is not related to the functionality of the dashboard itself.
thanks @iamsolankiamit
@iamsolankiamit @chrisys I think we also discussed an info button for linking to what account key is?
@rahul-thakoor yes we did, but we didn't finalise on the content/link
I guess this? https://boinc.bakerlab.org/rosetta/weak_auth.php
it asks you to login, do we have any link which is open and user can read it?
I'll take this up later today maybe in 5-6 hours. So if you want you can merge what we have now and i'll create another pr with that change.
@balena-ci retest
Visually and operationally this looks great to me, I defer to @rahul-thakoor and @sradevski to confirm their comments are addressed and then LGTM!