You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've reviewed the code in the main HTML file and would like to suggest a variable name change to improve code readability and understanding.
Current variable name: userHex
Suggested variable name: userPublicKey
The reason for this change is that the variable represents the public key of the user, and naming it userPublicKey would make the code more self-explanatory and easier to understand.
Here's the code snippet with the suggested change:
// Before changevaruserHex,filename,contentType,path;
...
asyncfunctiongenerateAuthorizationHeader(pubkey){
...
return`Nostr ${btoa(JSON.stringify(signedEvent))}`}...console.log(`Logged in with public key: ${userHex}`)...consttext=fileContent.valuevarauthorization=awaitgenerateAuthorizationHeader(userHex)...// After changevaruserPublicKey,filename,contentType,path;
...
asyncfunctiongenerateAuthorizationHeader(pubkey){
...
return`Nostr ${btoa(JSON.stringify(signedEvent))}`}...console.log(`Logged in with public key: ${userPublicKey}`)...consttext=fileContent.valuevarauthorization=awaitgenerateAuthorizationHeader(userPublicKey)...
The text was updated successfully, but these errors were encountered:
I've reviewed the code in the main HTML file and would like to suggest a variable name change to improve code readability and understanding.
Current variable name:
userHex
Suggested variable name:
userPublicKey
The reason for this change is that the variable represents the public key of the user, and naming it
userPublicKey
would make the code more self-explanatory and easier to understand.Here's the code snippet with the suggested change:
The text was updated successfully, but these errors were encountered: