Skip to content

fix: empty lens select#1247

Merged
CyberTimon merged 1 commit into
CyberTimon:mainfrom
lneugebauer:1245-fix-empty-lens-select
Jun 6, 2026
Merged

fix: empty lens select#1247
CyberTimon merged 1 commit into
CyberTimon:mainfrom
lneugebauer:1245-fix-empty-lens-select

Conversation

@lneugebauer
Copy link
Copy Markdown
Contributor

Description

The settings screen has the option to add lens shortcuts but does not populate the select.
This pull request adds back the lensfun db call that has been removed in fc0ca85.
Closes #1245.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Performance improvement
  • Code refactoring
  • Documentation update
  • UI/UX improvement
  • Build/CI or Dependency update

Changes Made

  • Add useEffect hook to get lensfun data.

Screenshots/Videos

Testing

  • I have tested these changes locally and confirmed that they work as expected without issues

Test Configuration:

  • OS: macOS Tahoe
  • Hardware: Apple M1 Max

Checklist

  • My code follows the project's code style
  • I haven't added unnecessary AI-generated code comments
  • My changes generate no new warnings or errors

Additional Notes

AI Disclaimer:

Please state the involvement of AI in this PR:

  • This PR is entirely AI-generated
  • This PR is AI-generated but guided by a human
  • This PR was handwritten with AI assistance (spell check, logic suggestions, error resolving)
  • This PR contains only blood, sweat, and coffee (AI-free)

@lneugebauer lneugebauer requested a review from CyberTimon as a code owner June 5, 2026 16:40
@CyberTimon
Copy link
Copy Markdown
Owner

Thanks a lot for the fix! I haven't noticed this when I refactored....

@CyberTimon CyberTimon merged commit 51081b4 into CyberTimon:main Jun 6, 2026
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.

BUG: Lens database is empty

2 participants