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

Invalid search term. 404 error. When trying to view second page of Approval drawer #6803

Closed
pjeanmonod opened this issue Apr 22, 2020 · 7 comments

Comments

@pjeanmonod
Copy link

ISSUE TYPE
  • Bug Report
SUMMARY

404 invalid search when trying to view second page of notification drawer.

ENVIRONMENT
  • AWX version: 11.0.0
  • AWX install method: docker on linux
  • Ansible version: 2.9.5
  • Operating System: Linux
  • Web Browser: Chrome
STEPS TO REPRODUCE

Go to second page of Approval notification drawer, get 404 error.

EXPECTED RESULTS

the second page of results.

ACTUAL RESULTS

Invalid search term entered. GET returned: 404 Invalid page.

ADDITIONAL INFORMATION

image

image

@pjeanmonod
Copy link
Author

image

@pjeanmonod
Copy link
Author

Bug is still unresolved, Closed in error.

@stefanmcshane
Copy link

Getting a similar error for notifications, that I assume is related. Means we cannot see anything past the first 5 jobs that require approval, which is a bit of a show stopper for production use.

  • AWX Version 11.0.0

On opening the notifications drawer, the following is executed:
GET /api/v2/workflow_approvals/?page=1&page_size=5&order_by=created&status=pending

On using the pagination buttons, whether it is a number, or an arrow, the following is executed:
GET /v2/unified_job_templates/?page=2&page_size=5&order_by=created&status=pending

I'm not a frontend developer, but think this could be related to:

@unlikelyzero
Copy link

Notification Tray Tests:

  • Verify that WFJT Approval details are accurate
  • Verify that WFJT Link navigates user to correct JT
  • User can click Approve button to approve a WFJT
  • User can click Deny button to deny a WFJT.
  • User can sort Approvals by Created Date
  • User can Use Pagination controls to find the 21st Approval WFJT and approve

@chbmckeown
Copy link

Having the same issue, to confirm was this issue identified and fixed by update?
Unsure from labels/comments...
Thanks

@pjeanmonod
Copy link
Author

The issue still persists and seems to be common... Has a fix been identified?

@pjeanmonod
Copy link
Author

pjeanmonod commented Jun 26, 2020

For anyone who finds themselves on this thread with the same issue, it has now been fixed in 11.2 as shown below:

commit b1b6af4
Author: John Mitchell [email protected]
Date: Fri Apr 24 15:31:20 2020 -0400

make sure right endpoint is getting called for notification paginationl

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

No branches or pull requests

6 participants