Skip to content

Add joblog in self view - #568

Merged
SanjulaGanepola merged 3 commits into
mainfrom
feat/JoblogSelf
Aug 17, 2026
Merged

Add joblog in self view#568
SanjulaGanepola merged 3 commits into
mainfrom
feat/JoblogSelf

Conversation

@buzzia2001

Copy link
Copy Markdown
Member

Changes

This PR allows you to view the job log directly from the SELF view.

How to test this PR

Connect to a system, open an .sql file, and enter an incorrect query to generate a log in SELF (make sure SELF is enabled).
In the error details table, click the job name or the info icon next to the job name; the job log option will open.

image

Checklist

  • have tested my change
  • have created one or more test cases
  • updated relevant documentation
  • Remove any/all console.logs I added
  • have added myself to the contributors' list in CONTRIBUTING.md

Closes #246

@buzzia2001 buzzia2001 self-assigned this Aug 5, 2026
@buzzia2001 buzzia2001 added the enhancement New feature or request label Aug 5, 2026
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

👋 A new build is available for this PR based on c689c4a.

@SanjulaGanepola SanjulaGanepola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@buzzia2001 When I run an debug the extension, the following contribution is removed:

Image

Not sure why exactly since I see it listed in the contributes.json file. Do you see this on your end?

@buzzia2001

Copy link
Copy Markdown
Member Author

@SanjulaGanepola copy and paste... in contributes.json there was two twins entries, same command, same when: one inline@1, one navigation. I deleted the navigation one, keep the inline@1

Comment thread src/views/jobManager/selfCodes/selfCodesResultsView.ts Outdated

@buzzia2001 buzzia2001 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch... not quite just SelfErrorJobItem though. The SelfCodeTreeItem is useless, but we need string beacuse the item's own command passes jobName directly, not the tree item itself

@SanjulaGanepola
SanjulaGanepola merged commit 4d7b770 into main Aug 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Button on job in SELF to show job log

2 participants