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

Fix a problem with overwriting messages with the same content (issue #107) #109

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

Conversation

coperius
Copy link
Contributor

@coperius coperius commented Aug 6, 2024

Fix a problem with overwriting messages with the same content (issue #107 )

  • change primary key for message table
  • add "deleted" field for message table to avoid retrying the processing
  • add migration for message table
  • add DeleteMessage function to mark processed and deleted messages
  • change Messages function to return all messages by msg
  • fix and add unit tests

@coperius coperius requested a review from umputun as a code owner August 6, 2024 14:59
@coperius coperius marked this pull request as draft August 6, 2024 14:59
@coperius coperius changed the title Fix/issue107 Fix a problem with overwriting messages with the same content Aug 6, 2024
…mputun#107 )

- change primary key for message table
- add "deleted" field for message table to avoid retrying the processing
- add migration for message table
- add DeleteMessage function to mark processed and deleted messages
- change Messages function to return all messages by msg
fix and add unit tests
@coperius coperius marked this pull request as ready for review August 6, 2024 15:11
@coperius coperius changed the title Fix a problem with overwriting messages with the same content Fix a problem with overwriting messages with the same content (issue #107) Aug 6, 2024
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.

1 participant