Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #3445957: Organization field doesn't appear on registration field settings #3895

Merged
merged 1 commit into from May 20, 2024

Conversation

navneet0693
Copy link
Collaborator

Problem

In PR #3799, we added a hook api from the registration module to not allow these organization fields to beadded to the registration.

Solution

We are reverting the same logic in this PR and it will be moved social_organization module.

Issue tracker

https://www.drupal.org/project/social/issues/3445957

Theme issue tracker

How to test

  • Enable the modules: Social Registration Fields
  • Clear cache
  • Go to: /admin/config/people/social-registration-fields
  • The Organization field should be showing up.

Screenshots

Before
Screenshot 2024-05-08 at 1 00 02 PM

After
Screenshot 2024-05-08 at 1 00 13 PM

Release notes

We added the organizations profile fields to be used for the user registration page.

Change Record

N.A

Translations

N.A

…egistration field settings

In PR #3799, we added a hook api from the registration module to not allow these organization fields to beadded to the registration.

We are reverting the same logic in this PR and it will be moved social_organization module.
@navneet0693 navneet0693 added type: bug Fixes a bug in Open Social status: needs review This pull request is waiting for a requested review team: guardians labels May 8, 2024
@navneet0693 navneet0693 added this to the 13.0.0-alpha2 milestone May 8, 2024
@navneet0693 navneet0693 changed the title Issue #3445957 by navneet0693: Organization field doesn't appear on registration field settings Issue #3445957: Organization field doesn't appear on registration field settings May 8, 2024
Copy link
Contributor

@vcsvinicius vcsvinicius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve

@navneet0693 navneet0693 merged commit 10ae5c6 into main May 20, 2024
193 checks passed
@navneet0693 navneet0693 deleted the issue/3445957-organization-field branch May 20, 2024 03:58
@navneet0693 navneet0693 removed the status: needs review This pull request is waiting for a requested review label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 participants