Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update timeout time #6001

Closed
wants to merge 2 commits into from
Closed

Update timeout time #6001

wants to merge 2 commits into from

Conversation

Corefix
Copy link
Contributor

@Corefix Corefix commented Aug 5, 2016

A problem will occur when you are adding any success message and the quickly changing page, fx adding a product to cart and then going to another page before the 3 seconds pass. This is especially true when dealing with production mode sites with all cache enable.

I'm not a Javascript wizard so not sure any time is needed at all. But it should definitely be decreased. If their have been any thoughts on why this is 3 seconds I would love to know it.

A problem will occur when you are adding any success message and the quickly changing page, fx adding a product to cart and then going to another page before the 3 seconds pass. This is especially true when dealing with production mode sites with all cache enable.
@vkorotun vkorotun added Component: Customer Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development improvement and removed Area: Frontend Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development labels Aug 5, 2016
Updated it a little I have experienced issues with having it be 50 MS aswell, this might need further refactoring
@Corefix
Copy link
Contributor Author

Corefix commented Aug 25, 2016

@piotrekkaminski this is not a feature request but a FE bug that happens when you move from page to page "to fast"

@piotrekkaminski
Copy link
Contributor

@Corefix yes! moved by mistake sorry

@sevos1984
Copy link
Contributor

This was already fixed. See commits 541ec80, 55a6e51, b4d3739, eec31f6, af2a832, d97ebe5, 3120b1e and 66e6ad6. Thanks.

@sevos1984 sevos1984 closed this Sep 29, 2016
magento-engcom-team pushed a commit that referenced this pull request Aug 10, 2020
[TSG] MC-35218: [MFTF] VerifyCategoryProductAndProductCategoryPartialReindexTest failed often (MC-11386)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants