Add support for multi-select fields in DiscourseConnect protocol

Falco · February 24, 2022, 6:22pm

Continuing the discussion from Custom field in discourseconnect:

While site admins can create multiselect fields in Admin → Customize → User Fields in Discourse, at the moment it’s not possible to populate those fields with multiple values via a DiscourseConnect payload.

The plan is to add support for array params in the SSO payload like:

custom.user_field_1[]=valueA&custom.user_field_1[]=valueB

This almost works but is garbled by this line:

github.com

discourse/discourse/blob/main/lib/discourse_connect_base.rb#L151

      
        
            
            
  def to_h
                payload = {}
            
            
    ACCESSORS.each do |k|
                  next if (val = public_send(k)) == nil
                  payload[k] = val
                end
            
            
    @custom_fields&.each do |k, v|
                  payload["custom.#{k}"] = v.to_s
                end
            
            
    payload
              end
            end

where the Array.to_s means it gets persisted as

 ['valueA', 'valueB'].to_s
=> "[\"valueA\", \"valueB\"]"

Restricting the persisted values to the existing options in the Discourse Admin screen is not necessary.

sam · February 24, 2022, 8:50pm

Love the syntax being proposed here go head and make the change to add support for arrays

Topic		Replies	Views
Multiple custom fields for topics similar to custom userfields Dev	1	225	April 8, 2024
Custom field in discourseconnect Support	7	1111	February 24, 2022
Discourse_api sync_sso custom_fields results in "custom.custom.<field>" in payload Bug	3	671	August 5, 2020
How make multiselect fields? Dev	5	566	August 27, 2019
Ability to Select Multiple Options on a Dropdown User Field UX	11	1721	January 30, 2019

Add support for multi-select fields in DiscourseConnect protocol

Related Topics