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 message TPMR to RunOnReceive directive #320

Merged
merged 1 commit into from
Jun 28, 2017

Conversation

tbitowt
Copy link
Contributor

@tbitowt tbitowt commented Jun 27, 2017

Adds TPMR variable to RunOnReceive event.

With this variable, it's able to determine if a message is delivery report, and if it is, it's containing TPMR of base message.

Tested on Ubuntu 16.04.2

@codecov
Copy link

codecov bot commented Jun 27, 2017

Codecov Report

Merging #320 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #320      +/-   ##
==========================================
+ Coverage   36.51%   36.51%   +<.01%     
==========================================
  Files         214      214              
  Lines       63844    63847       +3     
==========================================
+ Hits        23313    23315       +2     
- Misses      40531    40532       +1

@nijel nijel merged commit 5607ac2 into gammu:master Jun 28, 2017
nijel added a commit that referenced this pull request Jun 28, 2017
- add changelog entry
- document when variable was added

Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel
Copy link
Member

nijel commented Jun 28, 2017

Merged, thanks for your contribution!

I've applied following additional fixes:

@nijel nijel self-assigned this Jun 28, 2017
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.

None yet

2 participants