IncidentManagementSystems / ims-issue-tracker

Repository to track issues and work for Incident Management Systems.
0 stars 0 forks source link

Clicking Save 2x fast on iPad causes double Drill to create #81

Closed tfremming1 closed 5 years ago

Sikta commented 5 years ago

Ravi is working on adding the saving/loading spinner on all such operations. I think that will resolve this but don’t know how long before he is done. Will waiting for that be ok?

tfremming1 commented 5 years ago

Waiting is fine. Just logged for tracking

Sent from iPhone

On Feb 25, 2019, at 7:38 PM, Lee Olsen notifications@github.com wrote:

Ravi is working on adding the saving/loading spinner on all such operations. I think that will resolve this but don’t know how long before he is done. Will waiting for that be ok?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Sikta commented 5 years ago

Btw, Ravi has completed condensing the top menu to a hamburger when on a mobile device (not perfect but will do for now). He also completed making it so if you go directly to a url you should be logged in for, it'll redirect to the login and back to the page you bookmarked after login is complete. He's currently working on the loading/saving spinner then is going to start looking into adding SignalR so we can push to the browser from the API (updating training attendance status without refreshing, etc).

Lee

On Mon, Feb 25, 2019 at 8:06 PM Travis Fremming notifications@github.com wrote:

Waiting is fine. Just logged for tracking

Sent from iPhone

On Feb 25, 2019, at 7:38 PM, Lee Olsen notifications@github.com wrote:

Ravi is working on adding the saving/loading spinner on all such operations. I think that will resolve this but don’t know how long before he is done. Will waiting for that be ok?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/IncidentManagementSystems/ims-issue-tracker/issues/81#issuecomment-467263211, or mute the thread https://github.com/notifications/unsubscribe-auth/AIAsqnmEwQw2BQY1zmo5s1VhNkyc1pJPks5vRJajgaJpZM4bRHfR .

tfremming1 commented 5 years ago

One question. When you log in will it show the spinner. If you “wake up” the app there is some lag time. Would be nice to see something is happening etc.

Sent from iPhone

On Feb 25, 2019, at 8:10 PM, Lee Olsen notifications@github.com wrote:

Btw, Ravi has completed condensing the top menu to a hamburger when on a mobile device (not perfect but will do for now). He also completed making it so if you go directly to a url you should be logged in for, it'll redirect to the login and back to the page you bookmarked after login is complete. He's currently working on the loading/saving spinner then is going to start looking into adding SignalR so we can push to the browser from the API (updating training attendance status without refreshing, etc).

Lee

On Mon, Feb 25, 2019 at 8:06 PM Travis Fremming notifications@github.com wrote:

Waiting is fine. Just logged for tracking

Sent from iPhone

On Feb 25, 2019, at 7:38 PM, Lee Olsen notifications@github.com wrote:

Ravi is working on adding the saving/loading spinner on all such operations. I think that will resolve this but don’t know how long before he is done. Will waiting for that be ok?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/IncidentManagementSystems/ims-issue-tracker/issues/81#issuecomment-467263211, or mute the thread https://github.com/notifications/unsubscribe-auth/AIAsqnmEwQw2BQY1zmo5s1VhNkyc1pJPks5vRJajgaJpZM4bRHfR .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Sikta commented 5 years ago

All save/load calls (basically every call to the api) will show the spinner. A lot of times the response will be so fast that you probably won't even see it pop up.

On Mon, Feb 25, 2019 at 8:20 PM Travis Fremming notifications@github.com wrote:

One question. When you log in will it show the spinner. If you “wake up” the app there is some lag time. Would be nice to see something is happening etc.

Sent from iPhone

On Feb 25, 2019, at 8:10 PM, Lee Olsen notifications@github.com wrote:

Btw, Ravi has completed condensing the top menu to a hamburger when on a mobile device (not perfect but will do for now). He also completed making it so if you go directly to a url you should be logged in for, it'll redirect to the login and back to the page you bookmarked after login is complete. He's currently working on the loading/saving spinner then is going to start looking into adding SignalR so we can push to the browser from the API (updating training attendance status without refreshing, etc).

Lee

On Mon, Feb 25, 2019 at 8:06 PM Travis Fremming < notifications@github.com> wrote:

Waiting is fine. Just logged for tracking

Sent from iPhone

On Feb 25, 2019, at 7:38 PM, Lee Olsen notifications@github.com wrote:

Ravi is working on adding the saving/loading spinner on all such operations. I think that will resolve this but don’t know how long before he is done. Will waiting for that be ok?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub < https://github.com/IncidentManagementSystems/ims-issue-tracker/issues/81#issuecomment-467263211 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AIAsqnmEwQw2BQY1zmo5s1VhNkyc1pJPks5vRJajgaJpZM4bRHfR

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/IncidentManagementSystems/ims-issue-tracker/issues/81#issuecomment-467266183, or mute the thread https://github.com/notifications/unsubscribe-auth/AIAsqs6XOVzg9l2ur5PqXHrCbBKkJSSUks5vRJnYgaJpZM4bRHfR .