Skip to content

Form: allow browser-info collection to be configurable#23

Merged
jayaddison merged 3 commits intomainfrom
issue-21/configurable-browser-info-fields
Apr 6, 2026
Merged

Form: allow browser-info collection to be configurable#23
jayaddison merged 3 commits intomainfrom
issue-21/configurable-browser-info-fields

Conversation

@jayaddison
Copy link
Copy Markdown
Member

Describe the reason for these changes and the problem that they solve

As described in #21, some use cases (such as RecipeRadar) do not require all of the browser information fields that are currently retrieved by default. We can make the selection of relevant fields configurable as an option to the Feedback API.

Briefly summarize the changes

  1. Add a browserInfo constructor argument to the Form page -- with a default of all existing browser environment properties already collected.
  2. Only gather browser information for properties that have a truthy value in the Form instance's browserInfo properties.
  3. Allow consumers of the Feedback component to specify options.browserInfo to configure their preferred set of browser info fields to collect.

How have the changes been tested?

  1. Local development testing.

List any issues that this change relates to
Resolves #21.

@jayaddison jayaddison merged commit bd6c1cd into main Apr 6, 2026
@jayaddison jayaddison deleted the issue-21/configurable-browser-info-fields branch April 6, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Configuration: allow reducing the subset of browser information collected

1 participant