hale feat: Add paramter "resolvedepth" in UI when downloading data via WFS

Add in the dialog for the WFS GetCapability the possibility to choose if you want to add a resolveDepth for a request and how much would that be. Has been added a spinner to choose a number above 0 or the '*' for all the possible depths to be resolved.

ING-4129

May 15 '24 13:05 emanuelaepure10

The text for the buttons will be updated in the next days.

May 15 '24 13:05 emanuelaepure10

The commit message and the PR should also reference the related GitHub issue #1085

May 22 '24 20:05 florianesser

The commit message and the PR should also reference the related GitHub issue #1085

The closing keyword (e.g. "Closes" or "Fixes") is still missing (see discussion here)

May 23 '24 09:05 florianesser

@stempler I just update with rebase and suddenly the build doesn't pass anymore. Any thoughts on that could cause this? Thank you

May 27 '24 13:05 emanuelaepure10

@emanuelaepure10 I think you remember these kind of errors from setting up the GitHub Actions workflows.

Request to http://build-artifacts.wetransform.to/p2/mirror/2020-03-releases/plugins/org.eclipse.equinox.log.stream_1.0.200.v20191017-2055.jar failed, trying cache instead...
Some attempts to read artifact osgi.bundle,org.eclipse.equinox.log.stream,1.0.200.v20191017-2055 failed:
   An error occurred while transferring artifact canonical: osgi.bundle,org.eclipse.equinox.log.stream,1.0.200.v20191017-2055 from repository http://build-artifacts.wetransform.to/p2/mirror/2020-03-releases:
      download from http://build-artifacts.wetransform.to/p2/mirror/2020-03-releases/plugins/org.eclipse.equinox.log.stream_1.0.200.v20191017-2055.jar failed

...

An error occurred while installing the items
Installation failed.
	session context was:(profile=DefaultProfile, phase=org.eclipse.equinox.internal.p2.engine.phases.Install, operand=null --> [R]org.eclipse.equinox.log.stream 1.0.200.v20191017-2055, action=org.eclipse.equinox.internal.p2.touchpoint.eclipse.actions.InstallBundleAction).
	The artifact file for osgi.bundle,org.eclipse.equinox.log.stream,1.0.200.v20191017-2055 was not found.
[a76044af-dd05-42e5-950f-2a17029a4fd0][extension>org.eclipse.tycho:tycho-maven-plugin:3.0.1] An error occurred while installing the items

The issue is not really resolved, just usually/mostly works, at least when a build cache is available. I don't know what the reason is when it fails despite having access to the cache.

But as you see this is not a required check. If you need the artifacts build, you can open the details and trigger a rerun of the failed job.

May 27 '24 13:05 stempler

I think you remember these kind of errors from setting up the GitHub Actions workflows. I do remember. My has just cannot click the rebase and merge when above is something red :-D

May 27 '24 13:05 emanuelaepure10

Shouldn't PRs only be merged after the corresponding ticket is challenged? I'm wondering because the merge now closed #1085 even though the challenge on ING-4129 was not done yet.

May 27 '24 14:05 florianesser

Shouldn't PRs only be merged after the corresponding ticket is challenged

Yes, that would be the desired approach. @emanuelaepure10 maybe you remember when we discussed the autoclosing of issues here.

May 27 '24 14:05 stempler

@stempler I remember very well the discussion, but I see no conclusion. Really sorry but for me this

Using the Closes #issue notation would be beneficial here, because (1) references to the issues could be automatically added to the changelog and (2) issues would be automatically commented in which release they were included (see related code in semantic-release/github)

didn't sound like THE way to go ahead from then on. But if that was the intention, please accept my apologies, and now I will know how to act for the future. I would say that we should also add to the internal issue a note for the challenger to inform that the issue is ready to be challenged.

May 27 '24 21:05 emanuelaepure10

But if that was the intention, please accept my apologies, and now I will know how to act for the future.

@emanuelaepure10 No need to apology, I just wanted reference the discussion as a reminder on what was discussed.

As you already had added the closes keyword after the feedback by @florianesser I don't see the relevance of the quote you stated, since you basically already applied that "way" by using the keyword.

May 28 '24 10:05 stempler

:tada: This PR is included in version 5.2.0 :tada:

The release is available on:

v5.2.0
GitHub release

Your semantic-release bot :package::rocket:

Jun 19 '24 15:06 we-release[bot]