fix: Use actual computed value for the URL

[juliusknorr]

@datenangebot My guess is that you could not reproduce as you are using a http dev instance? It likely takes the other path which takes the url from the input instead of the computed which was called differently.

Fix #723

tables/src/modules/sidebar/sections/SidebarIntegration.vue

Lines 69 to 75 in ccd4229

	apiEndpointUrl() {
	const params = {
	elementId: this.activeElement.id,
	}
	const url = '/apps/tables/api/1/' + (this.isView ? 'views' : 'tables') + '/{elementId}'
	return window.location.protocol + '//' + window.location.host + generateUrl(url, params)
	},

[juliusknorr]

/backport to stable0.6

[enjeck]

Do we care "fixing" about all the warnings thrown by GitHub Actions / eslint and GitHub Actions / Nextcloud?

[juliusknorr]

Do we care "fixing" about all the warnings thrown by GitHub Actions / eslint and GitHub Actions / Nextcloud?

Not necessarily in this pull request as it is unrelated to the bugfix, but would be good to get those warnings addressed as well of course to have a clean linting/static analysis output. Feel free to dive into that and open a separate PR.

Just a note, the Static analysis / Nextcloud which runs Psalm is probably worth to wait until @datenangebot wrapped up his refactoring on the database columns logic from #498

[juliusknorr]

Rebased to have CI run again after #733

[enjeck]

This looks good to me. I left a few comments on #733

[juliusknorr]

Thanks, feel free to also press merge if approved and CI is happy :)