Make the coverage summary be what takes you to the diff viewer

[armenzg]

We added a feature that allows you to click on a row and it takes you to the diff viewer.
This was good when the columns did not have links because if a column has a link then we can't use that column to trigger the onClick event. The last column is the only one that currently works.

Let's change the on row click behaviour and make that last column be the link.

[matthewacha]

hey @armenzg, is this issue being worked on? I could take it on

[armenzg]

@matthewacha sure! Thank you!

[matthewacha]

@armenzg The application is failing to run locally. Is there something I may have skipped on set up

[armenzg]

@matthewacha what's the issue you're facing? what are the steps you took?

@marco-c has the app being used these days? It seems to even fail on production:
https://firefox-code-coverage.herokuapp.com/#/

[matthewacha]

After cloning the repo, I run yarn install and yarn start. The build succeeds but localhost:5000 just returns a blank page. There is an error caused by react-hot-loader

[armenzg]

Do you see this in your console:

I'm hitting the same issue; would you like to try fixing it? or do you want us to look into it?

[matthewacha]

Yes, it is what I see in the console. it's fine, you may look into the React-Hot-Loader issue first, in the meantime, I have excluded the <AppContainer> wrapper so that I can proceed with fixing the column link issue

[marco-c]

@marco-c has the app being used these days? It seems to even fail on production:

We haven't been using it for a while, no. The changeset viewer part is not as useful now as we upload coverage to Phabricator. The file viewer part would be useful, but it's not complete.

[armenzg]

I've landed a fix on master. Please let me know if you still hit the issue.

@matthewacha I'm going to be away for almost 2 months and I've spoken to marco and it would make more sense for you to look for other projects in here:
http://codetribute.mozilla.org

It seems that the backend for this frontend is going to be written again, thus, it makes not much sense to work on this frontend.

[matthewacha]

Ok @armenzg , please inform me if its possible to contribute to the backend, I would not mind taking on some tasks.

[armenzg]

That's a question for @marco-c ; Thanks for your willingness!

[marco-c]

@matthewacha unfortunately the backend is still at the design phase, but soon there will be good-first-issues. I will make sure they are filed in codetribute too!

[matthewacha]

@marco-c cool. I will be waiting