.github/pull_request_template.md

@@ -24,22 +24,14 @@ You can mention here anything that you think is relevant for this PR. Some examp
 
 ## Checklist
 
-Are your changes backwards compatible? Please answer below:
-
-<!-- put your answer about backwards compatibility here -->
+- [ ] Are your changes backwards compatible?
 
 <!--
 If your PR is not ready for review yet, please mark your PR as a draft.
 If changes were requested, request a new review when you incorporated the feedback.
 -->
-Did you QA this? Could we deploy this straight to production? Please answer below:
+- [ ] Did you QA this? Could we deploy this straight to production?
 
-<!-- put your answer about QA here -->
+- [ ] For frontend changes: Tested on mobile?
 
-For frontend changes: Tested on mobile? Please answer below:
-
-<!-- put your answer about mobile QA here -->
-
-Did you introduce any new environment variables? If so, call them out explicitly here:
-
-<!-- put your answer about env vars here -->
+- [ ] Did you introduce any new environment variables? If so, call them out explicitly in the PR description.

components/webln/index.js

@@ -7,11 +7,9 @@ import { LNCProvider, useLNC } from './lnc'
 
 const WebLNContext = createContext({})
 
-const isEnabled = p => [Status.Enabled, Status.Locked].includes(p?.status)
-
 const syncProvider = (array, provider) => {
   const idx = array.findIndex(({ name }) => provider.name === name)
-  const enabled = isEnabled(provider)
+  const enabled = [Status.Enabled, Status.Locked].includes(provider.status)
   if (idx === -1) {
     // add provider to end if enabled
     return enabled ? [...array, provider] : array
@@ -115,7 +113,7 @@ function RawWebLNProvider ({ children }) {
   }, [setEnabledProviders])
 
   return (
-    <WebLNContext.Provider value={{ provider: isEnabled(provider) ? { sendPayment: sendPaymentWithToast } : null, enabledProviders, setProvider }}>
+    <WebLNContext.Provider value={{ provider: provider ? { sendPayment: sendPaymentWithToast } : null, enabledProviders, setProvider }}>
       {children}
     </WebLNContext.Provider>
   )