Skip to content

Conversation

sausage-todd
Copy link
Contributor

@sausage-todd sausage-todd commented Sep 11, 2023

Otherwise it'll duplicate them inifinitely. Plus, this functionality doesn't matter anyway

Changes proposed ✍️

What

🤖 Generated by Copilot at f7f5854

Simplified query for autoPublish setting in conversation.repo.ts. Removed unused enabled column from conversationSettings table as part of a refactor.

🤖 Generated by Copilot at f7f5854

autoPublish query
simplified by removing
enabled column

Why

How

🤖 Generated by Copilot at f7f5854

  • Simplify query for autoPublish setting by removing unused enabled column condition (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

Otherwise it'll duplicate them inifinitely. Plus, this functionality
doesn't matter anyway
@sausage-todd sausage-todd merged commit 970d708 into main Sep 11, 2023
@sausage-todd sausage-todd deleted the conversation-settings-bug branch September 11, 2023 13:49
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