Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Sending a message from the Messages tab creates a message with uuid equal to database URL #3242

Closed
alxndrsn opened this issue Mar 15, 2017 · 2 comments
Closed

Sending a message from the Messages tab creates a message with uuid equal to database URL #3242

alxndrsn opened this issue Mar 15, 2017 · 2 comments
Labels
Type: Bug Fix something that isn't working as intended
Milestone
March 14th - Marc...

Comments

@alxndrsn
Copy link
Contributor

https://github.com/medic/medic-projects/issues/1643

The guilty code: https://github.com/medic/medic-webapp/blob/master/static/js/services/send-message.js#L166 -> https://github.com/medic/medic-webapp/blob/master/static/js/services/send-message.js#L133-L143
@alxndrsn alxndrsn added the Type: Bug Fix something that isn't working as intended label Mar 15, 2017
alxndrsn added a commit that referenced this issue Mar 15, 2017
@alxndrsn
Generate UUIDs for outgoing SMS using uuid npm module
840af2b 
Issue: #3242
@alxndrsn alxndrsn self-assigned this Mar 15, 2017
@alxndrsn alxndrsn mentioned this issue Mar 15, 2017
Closed
garethbowen added a commit that referenced this issue Mar 15, 2017
@garethbowen
Avoid pouch db uuid generator
b380c3b 
#3242
@garethbowen garethbowen mentioned this issue Mar 15, 2017
Merged
@garethbowen garethbowen self-assigned this Mar 15, 2017
garethbowen added a commit that referenced this issue Mar 15, 2017
@garethbowen
Avoid pouch db uuid generator
c5547ac 
#3242
@garethbowen garethbowen added this to the March 14th - March 28th milestone Mar 15, 2017
@garethbowen garethbowen removed their assignment Mar 15, 2017
@garethbowen
Copy link
Contributor

worker-pouch issue created: pouchdb-community/worker-pouch#34

@alxndrsn alxndrsn removed their assignment Apr 11, 2017
@ngaruko
Copy link
Contributor

ngaruko commented Jun 27, 2017

Does this need AT @alxndrsn . How do you reproduce/check this?

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
Type: Bug Fix something that isn't working as intended
Projects
None yet
Milestone
March 14th - March 28th
Development

No branches or pull requests
3 participants
@alxndrsn @garethbowen @ngaruko