-
Notifications
You must be signed in to change notification settings - Fork 275
feat(community): update romm questions #5115
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
base: master
Are you sure you want to change the base?
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -74,6 +74,49 @@ questions: | |
| default: "" | ||
| required: true | ||
| private: true | ||
| - variable: enable_rescan_on_filesystem_change | ||
| label: Enable Rescan on Filesystem Change | ||
| description: Automatically re-scan the library when filesystem changes are detected. | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: enable_scheduled_rescan | ||
| label: Enable Scheduled Rescan | ||
| description: Enable periodic scheduled re-scanning of the library. | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: scheduled_rescan_cron | ||
| label: Scheduled Rescan Cron | ||
| description: Cron expression for scheduled re-scanning (e.g. "0 3 * * *" for 3:00 AM daily). | ||
| schema: | ||
| type: string | ||
| default: "0 3 * * *" | ||
| show_if: [["enable_scheduled_rescan", "=", true]] | ||
| - variable: disable_download_endpoint_auth | ||
| label: Disable Download Endpoint Auth | ||
| description: Required for Tinfoil/WebRcade integration (https://docs.romm.app/latest/Integrations/Tinfoil-integration/) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: flashpoint_api_enabled | ||
| label: Enable Flashpoint API | ||
| description: Metadata for 180,000+ flash and browser-based games (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#flashpoint) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: hasheous_api_enabled | ||
| label: Enable Hasheous API | ||
| description: Free, open-source hash-based game matching (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#hasheous) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: hltb_api_enabled | ||
| label: Enable HowLongToBeat API | ||
| description: Game completion times for 84,000+ games (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#how-long-to-beat) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: igdb_client_id | ||
| label: IGDB Client ID | ||
| description: Generate an ID and SECRET for IGDB (https://api-docs.igdb.com/#account-creation) | ||
|
|
@@ -87,20 +130,94 @@ questions: | |
| type: string | ||
| default: "" | ||
| private: true | ||
| - variable: launchbox_api_enabled | ||
| label: Enable LaunchBox API | ||
| description: Community-driven metadata database (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#launchbox) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: mobygames_api_key | ||
| label: MobyGames API Key | ||
| description: Generate an API key for MobyGames (https://www.mobygames.com/info/api) | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| private: true | ||
| - variable: playmatch_api_enabled | ||
| label: Enable PlayMatch API | ||
| description: Hash-based matching service used with IGDB (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#playmatch) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: retroachievements_api_key | ||
| label: RetroAchievements API Key | ||
| description: Generate an API key from your RetroAchievements account (https://retroachievements.org/settings) | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| private: true | ||
| - variable: screenscraper_user | ||
| label: ScreenScraper Username | ||
| description: Username for ScreenScraper.fr (https://docs.romm.app/latest/Getting-Started/Metadata-Providers/#screenscraper) | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| - variable: screenscraper_password | ||
| label: ScreenScraper Password | ||
| description: Password for ScreenScraper.fr | ||
| schema: | ||
| type: string | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this shouldnt be visible if the user field is empty
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great point, fixing.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is pw required when username is provided? If thats the case, we should require it. (required:true) |
||
| default: "" | ||
| private: true | ||
| - variable: steamgriddb_api_key | ||
| label: SteamGridDB API Key | ||
| description: Generate an API key for SteamGridDB (https://github.com/rommapp/romm/wiki/Generate-API-Keys#steamgriddb) | ||
| description: Generate an API key for SteamGridDB (https://www.steamgriddb.com/profile/preferences/api) | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| private: true | ||
| - variable: oidc_enabled | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OIDC most of the times require slightly different setup for each provider which will be an ever growing setup screen, I'd suggest removing this.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The OIDC variables presented are fairly uniform across all OIDC providers. While there may be additional values for things like a "preferred username" or custom claims, those extended configurations have been left to "additional environment variables" specifically because we should only be prompting for commonly used environment variables.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I feel that someone configuring OIDC will probably want to setup those custom things like preferred username etc and having them split up would be more confusing. |
||
| label: Enable OIDC Authentication | ||
| description: Enable OpenID Connect (OIDC) for single sign-on (https://docs.romm.app/latest/Getting-Started/OIDC-Setup/) | ||
| schema: | ||
| type: boolean | ||
| default: false | ||
| - variable: oidc_provider | ||
| label: OIDC Provider Name | ||
| description: Name of the OIDC provider (e.g. Authelia, Authentik, PocketID, Zitadel, Keycloak) | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| show_if: [["oidc_enabled", "=", true]] | ||
| - variable: oidc_client_id | ||
| label: OIDC Client ID | ||
| description: Client ID for OIDC authentication. | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| show_if: [["oidc_enabled", "=", true]] | ||
| - variable: oidc_client_secret | ||
| label: OIDC Client Secret | ||
| description: Client secret for OIDC authentication. | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| private: true | ||
| show_if: [["oidc_enabled", "=", true]] | ||
| - variable: oidc_redirect_uri | ||
| label: OIDC Redirect URI | ||
| description: "Absolute redirect URI (e.g. https://romm.example.com/api/oauth/openid)" | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| show_if: [["oidc_enabled", "=", true]] | ||
| - variable: oidc_server_application_url | ||
| label: OIDC Server Application URL | ||
| description: Absolute URL of the OIDC server application. | ||
| schema: | ||
| type: string | ||
| default: "" | ||
| show_if: [["oidc_enabled", "=", true]] | ||
| - variable: additional_envs | ||
| label: Additional Environment Variables | ||
| schema: | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.