Skip to content

New view for WRKJOB#556

Open
buzzia2001 wants to merge 1 commit into
codefori:mainfrom
buzzia2001:main
Open

New view for WRKJOB#556
buzzia2001 wants to merge 1 commit into
codefori:mainfrom
buzzia2001:main

Conversation

@buzzia2001

Copy link
Copy Markdown
Member

Changes

With this PR, the view for WRKJOB has been updated to use the one from the FS extension.

How to test this PR

  1. Connect to a system and hover over the name of the database connection profile
image
  1. A new tab containing WRKJOB 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

@buzzia2001 buzzia2001 requested a review from a team June 29, 2026 20:39
@sebjulliand

Copy link
Copy Markdown
Member

@buzzia2001 I like the idea, but there are two suggestions I would make:

  • There is the SQL job from the core that could benefit from having a joblog viewer
    image
  • In this case, the joblog webview should be moved to the core, so there is no circular dependency between Core and FS

What do you think ?

@NicolasSchindler

Copy link
Copy Markdown

@sebjulliand @buzzia2001 and i are already on that :) I prepare a PR later for the Core. Also to include the JDBC configuration of the Mapepire job

@sebjulliand

Copy link
Copy Markdown
Member

@sebjulliand @buzzia2001 and i are already on that :) I prepare a PR later for the Core. Also to include the JDBC configuration of the Mapepire job

Ok, cool!
Be careful about putting the JDBC options in the Core. The core's job is not meant to be overly configurable since it's used for the core's processes. It would be a shame if users could break it by misconfiguring it.

@buzzia2001

Copy link
Copy Markdown
Member Author

Ciao @sebjulliand @NicolasSchindler ,

I'll I'll move the WRKJOB view into the core extension, but I also have to move the frontend, so it'll take a little while.

@sebjulliand

Copy link
Copy Markdown
Member

@buzzia2001 sure, no rush 😄

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants