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

[Feat] Dynamic Placeholder Substitution #17

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

VipinDevelops
Copy link
Collaborator

@VipinDevelops VipinDevelops commented Aug 16, 2024

Issue(s)

#16

Acceptance Criteria fulfillment

  • Implemented placeholder identification and substitution for receiver-specific information
  • Tested and validated the feature in different messaging scenarios.

Proposed changes (including videos or screenshots)

Screencast.from.2024-08-16.19-57-27.webm

@VipinDevelops
Copy link
Collaborator Author

Hey @hugocostadev Please review this after the #15 PR. it's on top of it.

@VipinDevelops
Copy link
Collaborator Author

I've added additional features to our system, allowing us to dynamically retrieve and display the username, name, email, and room details. This functionality is now working across various platforms, including Channel, Private groups, Direct Message, and Live Chat.

@hugocostadev
Copy link
Collaborator

All good, just missing the README update! ;D

@VipinDevelops
Copy link
Collaborator Author

Sure I'll update those

@VipinDevelops
Copy link
Collaborator Author

image
Hey @hugocostadev sorry for the delay I added the details about the dynamic value in the body placeholder and in README

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