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	102	April 8, 2024
Custom field in discourseconnect support	7	994	February 24, 2022
Discourse_api sync_sso custom_fields results in "custom.custom.<field>" in payload bug	3	626	August 5, 2020
How make multiselect fields? dev	5	527	August 27, 2019
Is it possible to create custom dropdown type user field in discourse? support user-custom-fields	6	1978	August 3, 2015

Add support for multi-select fields in DiscourseConnect protocol

Related Topics