877157 - Implement new VIVO SMS protocol for checking balance

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Cost Control, defect)

Description

[Salvador de la Puente González [:salva]]

VIVO has changed the protocol to check balance. It is yet based on SMS but now the message being sent must be created dynamically to include a timestamp and other variable data.

See details in bug 875330.

Comment 1

[Salvador de la Puente González [:salva]]

Attachment: New VIVO SMS protocol

Comment 2

[Jose Manuel Cantera]

Comment on attachment 755523 [details]
New VIVO SMS protocol

good work Salva

Comment 3

[Salvador de la Puente González [:salva]]

Master: d2ad0f1a5a6be8a2d6df721ba3c2b33e037c423b

Comment 4

[Salvador de la Puente González [:salva]]

Comment on attachment 755523 [details]
New VIVO SMS protocol

This can be considered as the follow up for bug 875695 so, the same reasons apply. Renominating to leo?

Bug caused by (feature/regressing bug #): bug 875330 - this is the actual implementation of the new Vivo protocol.
User impact if declined: high - the application will cease to work
Testing completed: yes - and new unit tests to verify the protocol added
Risk to taking this patch (and alternatives if risky): low - despite the bunch of code added. It has been tested.
String or UUID changes made by this patch: none

Comment 5

[John Ford [:jhford] CET/CEST Berlin Time]

I was not able to uplift this bug to v1-train.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1-train
  git cherry-pick -x -m1 d2ad0f1a5a6be8a2d6df721ba3c2b33e037c423b
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 6

[Salvador de la Puente González [:salva]]

v1-train: 5534304aee934055f08f21ce93261ba2a596516a