Closed jekram closed 7 years ago
I have done testing of all use cases for individual Chats and updated results in test plan https://docs.google.com/spreadsheets/d/15PkCYlSKery_sDgabEBjz0_fGjjrb2zEbskFK9c-LRw/edit?pli=1#gid=0 . During testing, i was not able to reproduce this issue from iOS to iOS . However, i found 2 bugs which i have opened on github 440 and 441.
I have added some log messages to catch and debug if this issue happens again. Can we close this for now?
@sumairasaeed I am assuming these logs are on the client side? can we have these with a global tag so we can turn on logging and turn off logging. As logs can slow the performance. If you are not understanding let's discuss
These logs are on client side. yes sir I have understood this . Currently these logs are without any global tag. Yes we can modify logs to be turned ON or OFF with a flag.
On Tue, Dec 13, 2016 at 9:09 PM, Cloudkibo notifications@github.com wrote:
@sumairasaeed https://github.com/sumairasaeed I am assuming these logs are on the client side? can we have these with a global tag so we can turn on logging and turn off logging. As logs can slow the performance. If you are not understanding let's discuss
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Cloudkibo/iOS/issues/437#issuecomment-266780936, or mute the thread https://github.com/notifications/unsubscribe-auth/AKbhp6m47zDqz8R2e9XxIeRPkYZK58h6ks5rHsNHgaJpZM4LF8qh .
We should have two flags, a global logging flag and if that is turned off then none of the logging is on. The other would be a local flag and so that is off then that that logging is off.
So assume we have logging in three functions A, B & C. Then we should have that following ability:
Make sence?
Yes understood better now.
On Wed, Dec 14, 2016 at 1:41 AM, Cloudkibo notifications@github.com wrote:
We should have two flags, a global logging flag and if that is turned off then none of the logging is on. The other would be a local flag and so that is off then that that logging is off.
So assume we have logging in three functions A, B & C. Then we should have that following ability:
- All logging off. (We turn Global logging off - it does not matter what the status of A, B or C)
- We want to log on all function then we would turn global logging on and also flag for A, B, & C
- We want subset of logging then global logging should be on, and let's say flag for A & B on.
Make sence?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Cloudkibo/iOS/issues/437#issuecomment-266855656, or mute the thread https://github.com/notifications/unsubscribe-auth/AKbhp2SLdkARhLJ6qm1fG8udlr65pkfpks5rHwMGgaJpZM4LF8qh .
@sumairasaeed
This problem is still there. I am removing the resolved status.
You need to jump on this problem right away.
Next look into the logs and not waste time about it.
I had removed the app on iOS and reinstalled it
Then I sent two message to Andriod which were sent properly
Then Andriod sent 1 message this was not seen by iOS (Issue)
Then I sent another message from iOS to Andriod which got there
See the screen shots below:
Looking into it right away
On Tue, Dec 20, 2016 at 8:19 PM, Cloudkibo notifications@github.com wrote:
@sumairasaeed https://github.com/sumairasaeed
This problem is still there. I am removing the resolved status.
You need to jump on this problem right away.
Next look into the logs and not waste time about it.
I had removed the app on iOS and reinstalled it
Then I sent two message to Andriod which were sent properly
Then Andriod sent 1 message this was not seen by iOS (Issue)
Then I sent another message from iOS to Andriod which got there
See the screen shots below:
[image: image] https://cloud.githubusercontent.com/assets/5336012/21355870/7d98f9f8-c684-11e6-8cb7-d8fbf9573de5.png
[image: image] https://cloud.githubusercontent.com/assets/5336012/21355905/a5d0feac-c684-11e6-845e-ad4cda814d41.png
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Cloudkibo/iOS/issues/437#issuecomment-268269495, or mute the thread https://github.com/notifications/unsubscribe-auth/AKbhp_MgM5g51gGxu-EoAWINtqqgAhl1ks5rJ_IMgaJpZM4LF8qh .
I sent more messages and they have been received and displayed properly. The previous message is still missing.
I removed and reinstalled the app and now the missing message is there.
From the paper trail what do we see:
When the initial message was sent:
Let me know what the iOS client is reporting: 1) Do we see a crime being received? 2) Do we see we iOS requested the data from the server 3) Do we see iOS getting the data 4) Do we see iOS wrote to the SQL Lite
What do we see from the Server: 1) Did the server send the notification? 2) Did the server received the API request for the data? 3) Did the server send the data?
Now between to iOS devices (First device is getting messages 2-3 mins delayed with NO CHIME, and the Second device working Properly.
Device # 1 with delayed messages with NO Chime Device 2 working properly
Reply to previous questions:
Let me know what the iOS client is reporting:
What do we see from the Server:
Did the server send the data? Yes
Okay sir. I was unable to work on this issue yesternight due to bad internet. Even now internet is bad. I am leaving for office in a while and will debug and fix it today.
On Wed, Dec 21, 2016 at 6:07 AM, Cloudkibo notifications@github.com wrote:
From the paper trail what do we see:
When the initial message was sent:
Let me know what the iOS client is reporting:
- Do we see a crime being received?
- Do we see we iOS requested the data from the server
- Do we see iOS getting the data
- Do we see iOS wrote to the SQL Lite
What do we see from the Server:
- Did the server send the notification?
- Did the server received the API request for the data?
- Did the server send the data?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Cloudkibo/iOS/issues/437#issuecomment-268405828, or mute the thread https://github.com/notifications/unsubscribe-auth/AKbhp0oawx4iXLsTdi2yH6TDseBFM9esks5rKHvigaJpZM4LF8qh .
@sumairasaeed what is the update here?
@sumairasaeed There has been no update for the last 2 days
Update on this task was discussed on slack. I have now implemented the changed suggested by @sojharo on https://github.com/Cloudkibo/CloudKibo/issues/483 .
Thanks - what release? Have you updated the iOS documentation with this decision and changes.
Yes updated this in documentation. It is part of release 0.1275 .
@sumairasaeed I sent the message from Andriod to iOS I heard the notification but when I looked for it was not there.