Skip to content

Conversation

@tanja3981
Copy link
Contributor

No description provided.

Copy link
Contributor

@PieterGit PieterGit left a comment

Choose a reason for hiding this comment

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

I reviewed this patch. Looks good to me, but have not tested it.
@sulkaharo or @jasoncalabrese can you also have a look at this PR?

@gonium
Copy link

gonium commented Apr 15, 2018

I ran into the same issue (invalid ID when AndroidAPS recorded a profile change) - this pull request fixed it.

@tanja3981
Copy link
Contributor Author

Would it be possible to get this bugfix merged into dev, as more and more people experience this problem?
If not, cause someone explain to me why the pull request isn't approved so that I could change whatever necessary?
@sulkaharo @jasoncalabrese

@nightscout nightscout deleted a comment Apr 22, 2018
@MilosKozak MilosKozak merged commit b25ca06 into nightscout:dev Apr 22, 2018
PieterGit added a commit to PieterGit/cgm-remote-monitor that referenced this pull request Jul 4, 2018
@tanja3981 tanja3981 deleted the bugfix/issue3462 branch December 11, 2021 22:37
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.

4 participants