Skip to content

Send full FCnt to AS#2898

Merged
rvolosatovs merged 2 commits intoTheThingsNetwork:masterfrom
rvolosatovs:fix/ns-as-fcnt
Jul 14, 2020
Merged

Send full FCnt to AS#2898
rvolosatovs merged 2 commits intoTheThingsNetwork:masterfrom
rvolosatovs:fix/ns-as-fcnt

Conversation

@rvolosatovs
Copy link
Copy Markdown
Contributor

Summary

Ref https://github.com/TheThingsIndustries/lorawan-stack-support/issues/64
Follow-up of #2876

Changes

  • Send full FCnt to AS

Testing

Unit tests

Regressions

None

Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Compatibility: The changes are backwards compatible with existing API, storage, configuration and CLI, according to the compatibility commitments in README.md. The target branch is set to master if the changes are fully compatible with existing API, database, configuration and CLI.
  • Documentation: Relevant documentation is added or updated.
  • Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.
  • Commits: Commit messages follow guidelines in CONTRIBUTING.md, there are no fixup commits left.

@rvolosatovs rvolosatovs added bug Something isn't working c/network server This is related to the Network Server c/application server This is related to the Application Server labels Jul 14, 2020
@rvolosatovs rvolosatovs added this to the July 2020 milestone Jul 14, 2020
@rvolosatovs rvolosatovs self-assigned this Jul 14, 2020
@rvolosatovs rvolosatovs merged commit 435c7fd into TheThingsNetwork:master Jul 14, 2020
@rvolosatovs rvolosatovs deleted the fix/ns-as-fcnt branch July 14, 2020 09:17
rvolosatovs added a commit to rvolosatovs/lorawan-stack-fork that referenced this pull request Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working c/application server This is related to the Application Server c/network server This is related to the Network Server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants