Skip to content

fix: telegram client duplicate function removal#1140

Merged
mcp97 merged 1 commit into
elizaOS:developfrom
azep-ninja:fix/duplicate-tg-funtions
Dec 16, 2024
Merged

fix: telegram client duplicate function removal#1140
mcp97 merged 1 commit into
elizaOS:developfrom
azep-ninja:fix/duplicate-tg-funtions

Conversation

@azep-ninja
Copy link
Copy Markdown
Contributor

Relates to:

N/A

Risks

Low - Removing duplicate function.

Background

What does this PR do?

What kind of change is this?

Bug fix - Removing duplicate _isMessageForMe private function in Telegram messages.ts file.

Documentation changes needed?

N/A

Testing

Where should a reviewer start?

Detailed testing steps

Review messageManager.ts in telegram-client to confirm only 1 private _isMessageForMe now exists with same functionality to avoid issues.

N/A

N/A

N/A

N/A

Copy link
Copy Markdown

@vyrnix9582046173 vyrnix9582046173 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@mcp97 mcp97 left a comment

Choose a reason for hiding this comment

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

LGTM

@mcp97 mcp97 merged commit 7d6d121 into elizaOS:develop Dec 16, 2024
@azep-ninja azep-ninja deleted the fix/duplicate-tg-funtions branch December 17, 2024 06:41
lalalune pushed a commit that referenced this pull request May 3, 2026
fix: telegram client duplicate function removal
FranceFlapjack pushed a commit to FranceFlapjack/eliza that referenced this pull request May 16, 2026
…ions

fix: telegram client duplicate function removal
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.

3 participants