[Bug?]: dbAuth userAttributes comment error

[pantheredeye]

What's not working?

After setting up dbAuth, the signupOptions code in api\src\functions\auth.js has an example user attribute that is commented out. Well, I needed that exact field, so I uncommented it and ran into an error on the next signup attempt. I ended up adding an underscore to match the props in the handler, changing it from name: userAttributes.name to name: _userAttributes.name and sign up worked.

I am not sure what the _ indicates, but it is in the params for the handler, so I am guessing it should be included the commented out line?

This search: https://github.com/search?q=repo%3Aredwoodjs%2Fredwood%20userAttributes.name&type=code has quite a few files show up... if its a matter of going through most of these and adding _, could we label this "Good first Issue?"

Generated Code:

    handler: ({
      username,
      hashedPassword,
      salt,
      userAttributes: _userAttributes,
    }) => {
      return db.user.create({
        data: {
          email: username,
          hashedPassword: hashedPassword,
          salt: salt,
          // name: userAttributes.name
        },
      })
    },

Working code with accompanying paragraph at the top:

  const signupOptions = {
    // Whatever you want to happen to your data on new user signup. Redwood will
    // check for duplicate usernames before calling this handler. At a minimum
    // you need to save the `username`, `hashedPassword` and `salt` to your
    // user table. `userAttributes` contains any additional object members that
    // were included in the object given to the `signUp()` function you got
    // from `useAuth()`.
    //
    // If you want the user to be immediately logged in, return the user that
    // was created.
    //
    // If this handler throws an error, it will be returned by the `signUp()`
    // function in the form of: `{ error: 'Error message' }`.
    //
    // If this returns anything else, it will be returned by the
    // `signUp()` function in the form of: `{ message: 'String here' }`.
    handler: ({
      username,
      hashedPassword,
      salt,
      userAttributes: _userAttributes,
    }) => {
      return db.user.create({
        data: {
          email: username,
          hashedPassword: hashedPassword,
          salt: salt,
          name: _userAttributes.name
        },
      })
    },
    ```

### How do we reproduce the bug?

_No response_

### What's your environment? (If it applies)

_No response_

### Are you interested in working on this?

- [ ] I'm interested in working on this

[ahaywood]

@pantheredeye I'm meeting with @dthyresson later today, I'll ask him about the _

[dthyresson]

@pantheredeye The _ means that the variable or argument is unused. It's needed as limiting and TS will warn that you have declared userAttributes but not used it anywhere in the function. It's to prevent extraneous or ... well .. unused variables.

redwood/packages/auth-providers/dbAuth/setup/src/templates/api/functions/auth.ts.template

Line 132 in 852e457

userAttributes: _userAttributes,

The templates sets it as _userAttributes so one knows it is available -- but since // name: userAttributes.name does not in fact use it TS and lining won't complain.

In you case, just change the argument to be without the underscore and then in the user create data refer to the variable as name: userAttributes.name so everything matches up.

[pantheredeye]

That all makes sense and is easy enough. I was really hoping that we could make a note of sorts to inform the user how to handle this by either adding the _ or removing the _. Being naive to this, I kept the _ because I did not want to conflict with any existing userAttributes somehow.
🤷‍♂️

[pantheredeye]

How do we close this? I do feel like this will trip someone else up, so I would like to have a note in the docs explaining this. If that is a no vote, then we can just close this issue, correct?

[dthyresson]

If anything I’d update the template to:

1. add comment note to be sure to correct the unused variable
2. Or add a typescript ignore to allow unused variables and remove the underscore

I do think the ide and typescript give rather informative messaging to let the developer implement correctly