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

Update Iindex TOML config file #587

Closed
josecelano opened this issue Jul 10, 2024 · 0 comments · Fixed by #588
Closed

Update Iindex TOML config file #587

josecelano opened this issue Jul 10, 2024 · 0 comments · Fixed by #588
Assignees
Labels
Dependencies Related to Dependencies
Milestone

Comments

@josecelano
Copy link
Member

Relates to: torrust/torrust-index#670

These two config option in the Index TOML config file have been removed:

[auth]
email_on_signup = "optional"

[mail]
email_verification_enabled = false

We need to replace them with:

[registration]
[registration.email]
#required = false
#verified = false
@josecelano josecelano added the Dependencies Related to Dependencies label Jul 10, 2024
@josecelano josecelano self-assigned this Jul 10, 2024
@josecelano josecelano added this to the v3.0.0 milestone Jul 10, 2024
josecelano added a commit to josecelano/torrust-index-gui that referenced this issue Jul 10, 2024
Relates to: torrust/torrust-index#670

These two config option in the Index TOML config file have been removed:

```toml
[auth]
email_on_signup = "optional"

[mail]
email_verification_enabled = false
```

We need to replace them with:

```toml
[registration]
[registration.email]
```
@josecelano josecelano linked a pull request Jul 10, 2024 that will close this issue
josecelano added a commit that referenced this issue Jul 10, 2024
fc19076 fix: [#587] update Index TOML config file (Jose Celano)

Pull request description:

  Relates to: torrust/torrust-index#670

  These two config option in the Index TOML config file have been removed:

  ```toml
  [auth]
  email_on_signup = "optional"

  [mail]
  email_verification_enabled = false
  ```

  We need to replace them with:

  ```toml
  [registration]
  [registration.email]
  ```

ACKs for top commit:
  josecelano:
    ACK fc19076

Tree-SHA512: d2f02d04c2d08ce3ba29ee5208d720d45db7b1dd376a5f31dc6b22fb6396c7ea9f34577072f56678a9bd77f9d2f75d3989842f7395af49c57c9b26438dca3f12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Related to Dependencies
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant