Skip to content

Fix issue 17621#17644

Open
ayatarek831 wants to merge 1 commit into
eclipse-theia:masterfrom
ayatarek831:ayatarek831_issue_17621
Open

Fix issue 17621#17644
ayatarek831 wants to merge 1 commit into
eclipse-theia:masterfrom
ayatarek831:ayatarek831_issue_17621

Conversation

@ayatarek831

@ayatarek831 ayatarek831 commented Jun 9, 2026

Copy link
Copy Markdown

What it does

fix issue 17621

How to test

Described in issue 17621

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

@ndoschek

Copy link
Copy Markdown
Member

Hi @ayatarek831, thanks for the contribution!

Before we can accept the PR, you'll need to sign the Eclipse Contributor Agreement (ECA). It's a one-time requirement for contributing to Eclipse Foundation projects.

To sign:

  1. Create an Eclipse Foundation account (if you don't have one yet) using the same email as your commits: https://accounts.eclipse.org/user/register
  2. Sign the ECA: https://accounts.eclipse.org/user/eca

TIA :)

@ndoschek ndoschek self-requested a review June 11, 2026 07:54
@ayatarek831

Copy link
Copy Markdown
Author

Hi @ndoschek,

Done, Please let me know if something else is required.

Thanks

@ndoschek

Copy link
Copy Markdown
Member

Hi again @ayatarek831, I triggered the revalidation but the ECA check still fails. Could you please double-check that this exact email is listed on your EF account and that the ECA is signed under that account?

See here the status of the validation: https://api.eclipse.org/git/eca/status/gh/eclipse-theia/theia/17644

TIA!

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

Labels

None yet

Projects

Status: Waiting on reviewers

Development

Successfully merging this pull request may close these issues.

Tree view toolbar actions lose DOM id after #16148

2 participants