Skip to content

Error nodes: improve text to include the reason for the failure #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
adietish opened this issue Mar 20, 2020 · 1 comment · Fixed by #654
Closed

Error nodes: improve text to include the reason for the failure #28

adietish opened this issue Mar 20, 2020 · 1 comment · Fixed by #654
Assignees
Milestone

Comments

@adietish
Copy link
Collaborator

adietish commented Mar 20, 2020

When resources cannot be loaded (bcs of ex. permissions or connection timeout), the error node only reports the failure, doesn't mention the reason. It should also mention why loading failed.

image

discussed in #23 (comment)

@adietish
Copy link
Collaborator Author

The exception bubbles up to TreeStructure.getChildElements(Any, ITreeStructureContribution).

    private fun getChildElements(element: Any, contribution: ITreeStructureContribution): Collection<Any> {
        return try {
            contribution.getChildElements(element)
        } catch (e: java.lang.Exception) {
            logger<TreeStructure>().warn(e)
            listOf(e)
        }
    }

adietish added a commit to adietish/intellij-kubernetes that referenced this issue Aug 21, 2023
adietish added a commit to adietish/intellij-kubernetes that referenced this issue Aug 21, 2023
adietish added a commit to adietish/intellij-kubernetes that referenced this issue Aug 21, 2023
adietish added a commit that referenced this issue Aug 23, 2023
@adietish adietish moved this to ✅ Done in IDE Cloudaptors Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant