Skip to content

feat: consume org from folderConfig [IDE-1368]#327

Merged
andrewrobinsonhodges-snyk merged 22 commits intomainfrom
feat/IDE-1368_consume_org_from_server
Nov 21, 2025
Merged

feat: consume org from folderConfig [IDE-1368]#327
andrewrobinsonhodges-snyk merged 22 commits intomainfrom
feat/IDE-1368_consume_org_from_server

Conversation

@bastiandoetsch
Copy link
Copy Markdown
Contributor

@bastiandoetsch bastiandoetsch commented Sep 29, 2025

Description

Provide description of this PR and changes, if linked Jira ticket doesn't cover it in full.

Checklist

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

@bastiandoetsch bastiandoetsch requested review from a team as code owners September 29, 2025 14:55
@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Sep 29, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-1368_consume_org_from_server branch from b4f7165 to fb63a37 Compare September 29, 2025 15:00
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

just some formatting and pmd fixes

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is where the project org is displayed and can be edited.

Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/preferences/PreferencesPage.java Outdated
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-1368_consume_org_from_server branch 2 times, most recently from af395ab to 34f7a15 Compare September 29, 2025 15:04
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just moved from LSConfigurationUpdate.Settings to here.

@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-1368_consume_org_from_server branch 3 times, most recently from 06ab458 to 2bd97c3 Compare September 29, 2025 15:09
@bastiandoetsch bastiandoetsch changed the title feat: consume org from folderConfig feat: consume org from folderConfig [IDE-1368] Sep 29, 2025
@acke acke force-pushed the feat/IDE-1368_consume_org_from_server branch 3 times, most recently from 995aa86 to 69d9741 Compare October 20, 2025 09:42
@acke acke force-pushed the feat/IDE-1368_consume_org_from_server branch from 31510c1 to 25418a1 Compare November 10, 2025 15:42
bastiandoetsch and others added 10 commits November 10, 2025 17:15
Added help text to Snyk Code Security checkbox: "Snyk Code scans must be
enabled for the organization to run."
Removed disableSnykCodeIfOrgDisabled() method and all calls to it
Snyk Code checkbox is always selectable, regardless of SAST/org status
Removed conditional logic in toggleIgnoresButtons() that hid filters
based on CCI status
Open Issues and Ignored Issues filters are always visible
Updated tooltips in plugin.xml to mention: "Note: This filter will only
take effect if Code Consistent Ignores (CCI) is enabled for the
organization."
@acke acke force-pushed the feat/IDE-1368_consume_org_from_server branch from 25418a1 to b2ad6a2 Compare November 10, 2025 16:15
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 13, 2025

CLA assistant check
All committers have signed the CLA.

var url = Preferences.getInstance().getEndpoint() + "/account";
var urlUserDocs = "https://docs.snyk.io/developer-tools/snyk-ide-plugins-and-extensions/eclipse-plugin/configuration-of-the-eclipse-plugin#global-settings";
addField(new LabelFieldEditor(
"Use automatic organization selection. When enabled, Snyk will automatically select the most\n"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We might still fall back to the top-level configured Org.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/properties/ProjectPropertyPage.java Outdated
Comment thread plugin/src/main/java/io/snyk/eclipse/plugin/properties/ProjectPropertyPage.java Outdated
@acke acke force-pushed the feat/IDE-1368_consume_org_from_server branch from e097d77 to 36a0a92 Compare November 14, 2025 11:54
@nick-y-snyk nick-y-snyk force-pushed the feat/IDE-1368_consume_org_from_server branch from 70a7529 to 7a1ffb9 Compare November 19, 2025 09:32
@nick-y-snyk nick-y-snyk force-pushed the feat/IDE-1368_consume_org_from_server branch from 7a1ffb9 to 833225b Compare November 19, 2025 09:37
@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk merged commit 5db1f52 into main Nov 21, 2025
8 of 9 checks passed
@andrewrobinsonhodges-snyk andrewrobinsonhodges-snyk deleted the feat/IDE-1368_consume_org_from_server branch November 21, 2025 17:33
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.

7 participants