Skip to content
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

Refactored the Image Uploader feature. #4971

Merged
merged 18 commits into from
Nov 19, 2023
Merged

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented Nov 18, 2023

Description

It's not an uploader only for i.nuuls.com anymore.
Changes:

  • Nuke unimplemented upload() overloads.
  • Move the entire code.
  • Make it into a singleton
  • Move message creation to MessageBuilder.

I still want to edit this branch and will force push :)

@Mm2PL Mm2PL force-pushed the chore/refactor_image_uploader branch from 42427bf to e3e169f Compare November 18, 2023 21:41
@Mm2PL Mm2PL force-pushed the chore/refactor_image_uploader branch from e3e169f to 5c00d0d Compare November 18, 2023 21:57
@Mm2PL Mm2PL changed the title Refactor: image uploader feature Refactored the Image Uploader feature. Nov 18, 2023
@Mm2PL Mm2PL force-pushed the chore/refactor_image_uploader branch from 9bbcfab to 69078d1 Compare November 18, 2023 22:03
@Mm2PL Mm2PL force-pushed the chore/refactor_image_uploader branch from 69078d1 to f045ae1 Compare November 18, 2023 22:04
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/messages/MessageBuilder.cpp Show resolved Hide resolved
src/messages/MessageBuilder.cpp Show resolved Hide resolved
src/messages/MessageBuilder.hpp Show resolved Hide resolved
Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM - Easy to review refactor

Merge it in when you've checked out my commits

@Mm2PL Mm2PL enabled auto-merge (squash) November 19, 2023 11:01
@Mm2PL
Copy link
Collaborator Author

Mm2PL commented Nov 19, 2023

image

@Mm2PL Mm2PL merged commit fbc8aac into master Nov 19, 2023
18 of 19 checks passed
@Mm2PL Mm2PL deleted the chore/refactor_image_uploader branch November 19, 2023 11: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.

2 participants