Nick Taylor comments

Results 252 comments of


                                            Nick Taylor

trafficstars

Bug: Inter font not loading on macOS Safari

@ahmaddonishyar, please see the contributing guidelines above.

fix: resolve a11y warnings for label-has-associated-control rule

Hi @shubhamchasing, just seeing if you're blocked on anything. No rush on this, just checking in.

fix: resolve a11y warnings for label-has-associated-control rule

> My apologies @nickytonline I will submit the changes before this weekend, thanks for offering help No need to apologize. 😎 I was just checking in.

fix: resolve a11y warnings for label-has-associated-control rule

`components/organisms/UserSettingsPage/developer-pack-form.tsx` no longer exists so if you merge latest into your branch, it should remove it. If you run into issues, delete the file in your PR.

fix: resolve a11y warnings for label-has-associated-control rule

> `components/organisms/InsightPage/InsightPage.tsx` I think this file is also not used anywhere, can you please confirm? @shubhamchasing, yes, the `components/organisms/InsightPage/InsightPage.tsx` can be removed as well.

feat: fixing - feedback - Default view for Stars and Forks graph should be time-series

I'm not sure where we landed on this @open-sauced/design @open-sauced/engineering.

feat: fixing - feedback - Default view for Stars and Forks graph should be time-series

I'm going to go ahead and close this for now. Feel free to reopen it if you want to continue working on this @SURAJ-SHARMA27.

Bug: Total stars and forks seem to be lagging.

I believe this is the ETL issue that has been mentioned before. cc: @jpmcb

fix: failing test for background functions

@Skn0tt, do you want us to take this over?

Feature: feedback - repo page charts should be embeddable

I had a chat with @brandonroberts about this yesterday. We're going to pair on it most likely. The TLDR; of the POC * Create a Netlify function that uses Playwright...