Skip to content

Conversation

@jvoisin
Copy link
Collaborator

@jvoisin jvoisin commented Aug 18, 2025

No description provided.

@fguillot fguillot requested a review from Copilot August 21, 2025 02:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a configurable preference to enable or disable Miniflux's API functionality. When disabled, API routes and related UI elements are hidden from users.

  • Adds a new API configuration option that defaults to enabled
  • Conditionally renders API-related UI elements and routes based on the configuration
  • Updates documentation to describe the new API configuration option

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
miniflux.1 Adds documentation for the new API configuration option
internal/config/options.go Defines the new enabledAPI configuration field and accessor method
internal/config/parser.go Adds parsing logic for the API configuration option
internal/http/server/httpd.go Conditionally registers API routes based on configuration
internal/ui/ui.go Conditionally registers API key management routes based on configuration
internal/ui/view/view.go Passes API enabled status to templates
internal/template/templates/common/settings_menu.html Conditionally displays API keys menu item

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jvoisin jvoisin force-pushed the disabled_api branch 2 times, most recently from 9f5aeb1 to 7a7aa6d Compare August 21, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants