Skip to content
This repository has been archived by the owner on Mar 27, 2019. It is now read-only.

Reset view and pagination #132

Closed
Garath620 opened this issue Jun 9, 2017 · 10 comments
Closed

Reset view and pagination #132

Garath620 opened this issue Jun 9, 2017 · 10 comments

Comments

@Garath620
Copy link

Hello @djenriquez,

I think I have found a bug with pagination:
After viewing or editing a secret, pressing Cancel or Save will reset the filter and pagination.
The current page will be lost and the interface will show a another random(?) page

@djenriquez
Copy link
Owner

Thanks @Garath620, looking into this now.

@djenriquez djenriquez self-assigned this Jun 14, 2017
@djenriquez djenriquez added the bug label Jun 14, 2017
@djenriquez
Copy link
Owner

@Garath620 This fix is on :latest and :2.2.0, please verify and close this issue once you're confirmed the fix.

@Garath620
Copy link
Author

Garath620 commented Jun 15, 2017

Hi @djenriquez,
I have tested 2.2.0. Filter stay applied after viewing secret. Thanks
But without filter, I have the same behavior, the current page is lost and the interface show a another random.

@djenriquez
Copy link
Owner

Interesting, I thought I had tested that out. I'll take a look at it later tonight again.

@djenriquez
Copy link
Owner

@Garath620 I'm trying to duplicate the issue that you're describing but cannot. On 2.2.0, if I add a filter to the secrets list with multiple pages, verify it still has multiple pages, go to a random page, click out of the secret either with cancel or save, it brings me back to the correct page with the secret I just viewed/updated.

Without a filter, the behavior is exactly the same.

Can you provide any more clues to help me replicate the issue so that it can be fixed?

@Garath620
Copy link
Author

Hi @djenriquez,
You do not need to add a filter. Just select a secret name to view it. Press Cancel and the current page is changed.

@djenriquez
Copy link
Owner

I'll take another look. 👍

@Garath620
Copy link
Author

Hi @djenriquez
To help you duplicate the issue, I make a screencast
I have a generic backend test => for i in {1..100} \ ; do vault write test/test$i key=value; done
Look the current page number
vault-ui_reset_view

@djenriquez
Copy link
Owner

@Garath620 PTAL at image djenriquez/vault-ui:550de88 which has the changes needed to fix this issue. I'll merge this in as soon as I get a few eyes on #175.

@Garath620
Copy link
Author

Hi @djenriquez,
I confirm, this problem is solved
Thank you very much!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants