Conversation
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
2aba539 to
fe76fe7
Compare
|
@copilot can be shown the author and the last editor on interface? also can we see the history? and allow viewing diff? also, I had no interface for adding or viewing comments |
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Done in The KB UI now shows both the original author and the last editor, adds direct access to native WordPress comments, and exposes revision history/diff from the custom KB list/tree and the article view modal. I also added KB modal metadata for author/last editor, comment count, and revision count. Preview screenshot: https://github.com/user-attachments/assets/adda0c1e-19a6-4d34-8756-fabe70ff437d Note: I could not run the WP containerized tests in this sandbox because |
|
@copilot fix tests, the comments interface should be integrated, like in tasks, and call the rest api or whatever, but not going directly to cpt admin |
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Done in KB comments are now integrated in the article modal instead of linking to the CPT admin screen: there is an in-app Comments tab with the comment list, composer, and delete flow using the WordPress comments REST API, similar to tasks. I also updated the KB tests to match the new behavior and extended REST comment protection to cover Screenshot: https://github.com/user-attachments/assets/5c92fe35-8f76-49be-b13f-d7e7154a5f0f I could not run the PHPUnit suites to completion in this sandbox because the Composer test dependency |
|
@copilot messaging.ts:105 Event handler of 'message' event must be added on the initial evaluation of worker script. |
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Fixed in I addressed both issues: the Screenshot: https://github.com/user-attachments/assets/5c92fe35-8f76-49be-b13f-d7e7154a5f0f Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot messaging.ts:105 Event handler of 'message' event must be added on the initial evaluation of worker script. |
…_post_revisions()) Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
…post-install Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Fixed in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
wp_count_post_revisions()is not a WordPress core function, causing fatal errors in both CI and Playgroundcount( wp_get_post_revisions() )which is the standard WordPress APIOriginal prompt
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.