Problem

There is no schema declaration for the social_profile.settings config object.

Proposed solution

Add a config/schema/social_profile.schema.yml file with the following contents:

social_profile.settings:
  type: config_object
  label: 'Social Profile Settings'
  mapping:
    social_profile_show_email:
      type: boolean
      label: 'Whether to show e-mail addresses on user profiles.'

Needed discussion

Discuss whether the labels/descriptions are correct.

CommentFileSizeAuthor
#4 2947921-4.patch432 bytesrajeshwari10

Comments

Kingdutch created an issue. See original summary.

kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Issue tags: +Novice
rajeshwari10’s picture

rajeshwari10 commented
Assigned: Unassigned » rajeshwari10
rajeshwari10’s picture

rajeshwari10 commented
StatusFileSize
new 2947921-4.patch432 bytes

Created config/schema/social_profile.schema.yml file and added the above-said content in it.

Please Review.

Thanks

rajeshwari10’s picture

rajeshwari10 commented
Assigned: rajeshwari10 » Unassigned
Status: Active » Needs review
kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Pull request for this has been created here: https://github.com/goalgorilla/open_social/pull/759

jaapjan’s picture

jaapjan commented
Status: Reviewed & tested by the community » Fixed

Thanks all!

  • Kingdutch committed 1d80d58 on 8.x-2.x 
    Issue #2947921 by rajeshwari10, Kingdutch: Missing schema for...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.