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

feat: add feature to configure module automatically on install #1

Merged
merged 2 commits into from
Jul 6, 2024

Conversation

md61421
Copy link
Owner

@md61421 md61421 commented Jul 4, 2024

Overview

This pull request (PR) introduces an automated installation script for the supabase_authentication module in Drupal. The script handles the configuration of user registration settings and form display fields. Additionally, the email_registration module is set as a dependency.

Before

Before this PR, the supabase_authentication module required manual configuration steps for user registration settings and form display fields.

Screenshots:

image
User registration settings needed to be configured manually.

image
Form display fields had to be manually adjusted.

After

After this PR, the configuration process is automated via the installation script.
Screenshots:

image
User registration settings are now configured automatically.

image
Form display fields are adjusted automatically.

Technical Details

  • Automated Configuration: The installation script configures user registration settings to allow visitors to register without email verification.
  • Form Display Adjustment: The script modifies the user form display to only include the "Username and password" field, removing all other fields.
  • Dependency Added: The email_registration module is set as a dependency in the supabase_authentication.info.yml file.

@ayushmishra206
Copy link
Collaborator

Changes look good, lets fix the linter issues. Thanks!

@ayushmishra206
Copy link
Collaborator

Great work on adding the PR description!

@md61421 md61421 force-pushed the feature branch 2 times, most recently from c55d7f3 to 835b8f5 Compare July 6, 2024 09:55
@md61421 md61421 merged commit 3be470b into main Jul 6, 2024
2 checks passed
@md61421 md61421 deleted the feature branch July 6, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants