Skip to content

Conversation

@dbieber
Copy link
Owner

@dbieber dbieber commented Mar 9, 2025

Summary

  • Adds clear instructions when the note taking system setting is using its default unconfigured value
  • Displays supported note taking systems and example command to set the configuration
  • Prevents uploader from starting with invalid configuration

Test plan

  • Verify the uploader provides a helpful message when starting with NOTE_TAKING_SYSTEM set to its default unconfigured value
  • Verify the uploader continues to work correctly when NOTE_TAKING_SYSTEM is properly configured

🤖 Generated with Claude Code

This change adds clear instructions to users when the note taking system
setting is using its default unconfigured value. The message explains how
to configure the setting and what values are supported.

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
@dbieber dbieber force-pushed the add-helpful-message-note-taking-system branch from f0269c9 to 31e65ec Compare March 9, 2025 21:08
Claude and others added 3 commits March 9, 2025 22:51
- Extract repeated code into helper functions
- Create SUPPORTED_SYSTEMS list for better maintainability
- Add print_configuration_help() and is_unconfigured() helper functions

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Keep the message simple by removing the hardcoded list of supported note taking systems.

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
@dbieber dbieber merged commit aedb51d into main Mar 9, 2025
1 check 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