github-project icon indicating copy to clipboard operation
github-project copied to clipboard

Pass `field` context as third argument to `matchFieldOptionValue()`

Open gr2m opened this issue 3 years ago • 0 comments

Please avoid duplicates

What’s missing?

The current matchFieldOptionValue constructor options receives two arguments

function (fieldOptionValue, userValue) {
  // return true or false
}

I suggest we pass a 3rd context argument

function (fieldOptionValue, userValue, context) {
  // return true or false
}

context would look like this

{
  field: {
    name: "My Field",
    userName: "myField",
    // maybe also `type`, and `options`, but we could add that later
  }
}

That way we could add more context in future, e.g. project, which would be the current project instance.

Why?

The way options are matched might differ depending on the field

Alternatives you tried

:shrug

Would you be interested in contributing the feature?

  • [ ] yes

gr2m avatar Mar 27 '22 18:03 gr2m