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

fix ActivityStream button at details page #15286

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

aeter
Copy link
Contributor

@aeter aeter commented Jun 20, 2024

SUMMARY

fixes bug:

  • when at the /details page for any item (host, project, schedule, etc.) and clicking on the Activity Stream button at the top right:
    ** there is not shown an Activity Stream specific to the item, but a general Activity Stream for the item category (schedules, templates, hosts, projects, inventories, etc.)

This PR adds a item__id=N to the Activity Stream button's URL from /details. Like: host__id=2 or credential__id=13 or project__id=5, etc. - thus filtering to the specific item's ActivityStream.

(for 03771872)

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • UI
AWX VERSION
awx: 0.1.dev34000+gbdfdbf6.d20240620
ADDITIONAL INFORMATION

I'm sorry, I could not find a shorter way to fix this - resulting in a rather big PR. If you could spot a better way for fixing it, let me know, and I'll be happy to discuss and rework the code. Thank you.

activitystream1


activitystream2


activitystream3


Ran npm tests, passed.

npm run test
npm run prettier

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

1 participant