Skip to content

Conversation

@cxyfer
Copy link
Owner

@cxyfer cxyfer commented Jul 5, 2025

Summary

  • Add optional public parameter to /daily, /daily_cn, /problem, and /recent commands
  • Allow users to control whether command responses are shown publicly or privately
  • Default behavior remains private (ephemeral messages) for backward compatibility

Changes

  • Added public: bool = False parameter to slash commands
  • Updated parameter descriptions to explain the new option
  • Modified send_daily_challenge in schedule_manager_cog to support ephemeral parameter
  • All ephemeral flags now use ephemeral=not public pattern

Test plan

  • Test /daily command with default (private) response
  • Test /daily command with public=True
  • Test /daily_cn command with both private and public options
  • Test /problem command with both private and public options
  • Test /recent command with both private and public options
  • Verify error messages respect the public parameter setting
  • Ensure scheduled daily challenges (non-interaction) still work

🤖 Generated with Claude Code

Add optional `public` parameter to /daily, /daily_cn, /problem, and /recent commands to control whether responses are shown publicly or privately (ephemeral).

- Default behavior remains private (ephemeral=True)
- Users can set public=True to share responses publicly
- Update send_daily_challenge in schedule_manager_cog to support ephemeral parameter

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@cxyfer cxyfer merged commit e9dbe8b into main Jul 5, 2025
@cxyfer cxyfer deleted the feat/daily branch July 5, 2025 08:05
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.

1 participant