FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix: AI Request Logs: "Copy Log ID" gives no feedback when copied by hbhalodia · Pull Request #700 · WordPress/ai · GitHub

/ ai Public

Fix: AI Request Logs: "Copy Log ID" gives no feedback when copied - #700

Merged
dkotter merged 2 commits into
WordPress:developfrom
hbhalodia:fix/issue-699
Jun 15, 2026
ConversationCommits (2)ChecksFiles changed
Merged

Fix: AI Request Logs: "Copy Log ID" gives no feedback when copied#700
dkotter merged 2 commits into
WordPress:developfrom
hbhalodia:fix/issue-699

Conversation

hbhalodia commented Jun 11, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

What?

Closes #699

Why?

  • PR resolves the accessibility issue in AI Request logs details modal.
  • UI does show if the log id has been copied or not.

How?

  • Adds the new hook to consolidate functionality of copy into single hook.
  • Add the functionality to request logs feature and update to use hook in meta description.

Use of AI Tools

  • Yes, Claude Code, Opus 4.8
  • Used for Drafting issue description, and provided idea on how to implement and code was written by AI.
  • Implementation was tested and reviewed by me.

Testing Instructions

  1. Enable request logging and generate at least one AI request so a log entry exists.
  2. Open AI → Request Logs and click a log row to open the detail modal.
  3. Click the Copy Log ID button in the modal footer.
  4. Observe the button.
  5. It should change the text, and for accessibilty it should speak as well.

Screenshots or screencast

Screen.Recording.2026-06-11.at.4.47.28.PM.mov

Changelog Entry

Fixed - AI Request Logs: "Copy Log ID" gives no feedback when copied. Updated the button text on copy.

github-actions Bot commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: hbhalodia <hbhalodia@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

dkotter commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

@hbhalodia A recent PR was merged that made some changes to the copy functionality in the Meta Description components. There's some merge conflicts on this PR that need addressed and we'll want to ensure we don't lose those recent changes

dkotter added this to the 1.2.0 milestone Jun 11, 2026

hbhalodia commented Jun 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Thanks @dkotter, I will update it and make sure it works as expected.

Update: The PR is now ready to review and resolved the conflicts.

dkotter modified the milestones: 1.2.0, 1.1.0 Jun 15, 2026
dkotter merged commit 4ff29b0 into WordPress:develop Jun 15, 2026
21 of 22 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

dkotter dkotter approved these changes

Assignees

hbhalodia

Labels

None yet

Projects

None yet

Milestone

1.0.2

Development

Successfully merging this pull request may close these issues.

AI Request Logs: "Copy Log ID" gives no feedback when copied

2 participants

Back | FazBrowse Home | New Git URL