Merge pull request #4743 from nextcloud-libraries/fix/ncuser-bubble

fix: NcUserBubble properties

src/components/NcUserBubble/NcUserBubble.vue

@@ -36,7 +36,7 @@ This component has the following slot:
 
 ```vue
 <p>
-	Some text before <NcUserBubble user="admin" display-name="Admin Example" :url="'/test'">@admin@foreign-host.com</NcUserBubble> and after the bubble.
+	Some text before <NcUserBubble user="admin" display-name="Admin Example" url="/test">@admin@foreign-host.com</NcUserBubble> and after the bubble.
 	<NcUserBubble avatar-image="icon-group" display-name="test group xyz" :primary="true">Hey there!</NcUserBubble>
 </p>
 ```
@@ -58,7 +58,15 @@ This component has the following slot:
 	</template>
 </NcUserBubble>
 </template>
-
+<script>
+export default {
+	methods: {
+		alert() {
+			alert('Removed')
+		},
+	},
+}
+</script>
 <style>
 .icon-close {
 	display: block;
@@ -115,6 +123,7 @@ This component has the following slot:
 import NcUserBubbleDiv from './NcUserBubbleDiv.vue'
 import NcAvatar from '../NcAvatar/index.js'
 import NcPopover from '../NcPopover/index.js'
+import Vue from 'vue'
 
 export default {
 	name: 'NcUserBubble',
@@ -143,7 +152,7 @@ export default {
 		 */
 		displayName: {
 			type: String,
-			required: true,
+			default: undefined,
 		},
 		/**
 		 * Whether or not to display the user-status
@@ -158,10 +167,10 @@ export default {
 		url: {
 			type: String,
 			default: undefined,
-			validator: url => {
+			validator: (url) => {
 				try {
-					url = new URL(url)
-					return !!url
+					url = new URL(url, url?.startsWith?.('/') ? window.location.href : undefined)
+					return true
 				} catch (error) {
 					return false
 				}
@@ -270,6 +279,11 @@ export default {
 			}
 		},
 	},
+	mounted() {
+		if (!this.displayName && !this.user) {
+			Vue.util.warn('[NcUserBubble] At least `displayName` or `user` property should be set.')
+		}
+	},
 	methods: {
 		onOpenChange(state) {
 			this.$emit('update:open', state)

src/globals.d.ts

@@ -0,0 +1,6 @@
+declare const PRODUCTION: boolean
+
+declare const SCOPE_VERSION: string
+
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+declare const TRANSLATIONS: { locale: string, translations: any }[]