Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Conversation

@dev-737
Copy link
Member

@dev-737 dev-737 commented Apr 7, 2024

TL;DR

Fabricating new messages based on user input and target, incorporating image URLs and censoring. Also, refactored code for vote incrementation.

What changed?

  • Added functions to get image URLs and build new embeds
  • Refactored code for vote incrementation
  • Updated checks in NetworkManager

How to test?

  • Test fabricating new messages with different input messages and targets
  • Test voting system with different users

Why make this change?

  • Improve message editing functionality
  • Refactor vote incrementation for better readability and efficiency

Copy link
Member Author

dev-737 commented Apr 7, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dev-737 and the rest of your teammates on Graphite Graphite

@deepsource-io
Copy link

deepsource-io bot commented Apr 7, 2024

Here's the code health analysis summary for commits 1ad02a1..f09a445. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
🎯 1 occurence resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@dev-737 dev-737 added the refactor something that needs a code refactor label Apr 7, 2024 — with Graphite App
@graphite-app
Copy link

graphite-app bot commented Apr 7, 2024

Graphite Automations

"Auto-assign PRs to author" took an action on this PR • (04/07/24)

1 assignee was added to this PR based on 's automation.

@dev-737 dev-737 merged commit 8ddf6f2 into main Apr 7, 2024
@dev-737 dev-737 deleted the 04-07-refactor_editmsg_decrease_code_complexity branch April 7, 2024 04:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

refactor something that needs a code refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants