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

Refactor Configuration for Filament Profile Editing and Avatar Handling #59

Merged
merged 5 commits into from
Dec 5, 2024

Conversation

ludndev
Copy link
Contributor

@ludndev ludndev commented Dec 4, 2024

Changes Introduced:

  1. Default Configuration Update:

    • Added a default configuration in filament-edit-profile.php.
    • Updated the README.md with configuration examples for clarity.
  2. Filesystem Configuration:

    • Now uses the FILESYSTEM_DISK environment variable to match the application's default filesystem disk.
    • Set fallback to 'public' instead of 'local' to prevent breaking changes in existing installations.
    • Configured default visibility as 'public' with comments explaining it can be customized if needed.
  3. Avatar Configuration:

    • Introduced configuration for specifying the avatar column name, defaulting to 'avatar_url'.
    • Updated the README.md to reflect this new configuration option.

… update README.md

- use of FILESYSTEM_DISK for disk to match default application filesystem disk.
- use of fallback 'public' and not 'local' to avoid introducing breaking changes in existing installation
- 'visibility' is 'public' while comment added to explain it can be updated
- update `README.md` config's example
- use of fallback 'avatar_url' to avoid introducing breaking changes in existing installation
- update example in `README.md`
refactor(config): add default config in filament-edit-profile.php and…
refactor(): add config for avatar. column name
@ludndev
Copy link
Contributor Author

ludndev commented Dec 4, 2024

@joaopaulolndev , could review please ? This PR address "Bug]: Possibilidade de inserir o nome da coluna do avatar #57"

@joaopaulolndev joaopaulolndev merged commit 990f7d1 into joaopaulolndev:main Dec 5, 2024
21 checks passed
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.

2 participants