While reviewing #910, I took a look at how the keys resource is used in various components. I noticed a line that uses keys and has a small issue. It checks whether keys is null, but because keys is a resource, it can never be null. I replaced the null check with a check that keys has data.
Before submitting this PR, please make sure you have:
[x] run npm run test and npm run lint and confirmed all checks still pass OR confirm CircleCI build passes
[x] verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced
While reviewing #910, I took a look at how the
keysresource is used in various components. I noticed a line that useskeysand has a small issue. It checks whetherkeysisnull, but becausekeysis a resource, it can never benull. I replaced thenullcheck with a check thatkeyshas data.Before submitting this PR, please make sure you have:
npm run testandnpm run lintand confirmed all checks still pass OR confirm CircleCI build passes