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

add MessageID and TimeStamp on NewsletterMessage #713

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

Conversation

RomySaputraSihananda
Copy link

Add MessageID and TimeStamp to NewsletterMessage struct

Summary

This pull request introduces the MessageID and TimeStamp fields into the NewsletterMessage struct. The new fields are essential for capturing the message's unique identifier and timestamp, ensuring better tracking and management of the newsletter messages.

Changes

  • Added MessageID field of type MessageID to the NewsletterMessage struct.
  • Added TimeStamp field of type time.Time to the NewsletterMessage struct.
  • Updated the creation of NewsletterMessage instances to include these new fields, using the id and t attributes respectively.

Copy link

@mateusfmello mateusfmello left a comment

Choose a reason for hiding this comment

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

All very well

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