refactor: remove default where a prop is required

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Author: susnux

src/components/NcActionLink/NcActionLink.vue

@@ -133,7 +133,6 @@ export default {
 		 */
 		href: {
 			type: String,
-			default: '#',
 			required: true,
 			validator: (value) => {
 				// href is either an anchor or a valid url starting with a scheme or a relative path

src/components/NcActionRouter/NcActionRouter.vue

@@ -74,7 +74,6 @@ export default {
 		 */
 		to: {
 			type: [String, Object],
-			default: '',
 			required: true,
 		},
 	},

src/components/NcAppSidebar/NcAppSidebar.vue

@@ -800,7 +800,6 @@ export default {
 		 */
 		name: {
 			type: String,
-			default: '',
 			required: true,
 		},
 

src/components/NcRichContenteditable/NcRichContenteditable.vue

@@ -332,11 +332,10 @@ export default {
 		},
 
 		/**
-		 * The content
+		 * The text content
 		 */
 		modelValue: {
 			type: String,
-			default: '',
 			required: true,
 		},
 

tests/unit/components/NcRichContenteditable/NcRichContenteditable.spec.ts

@@ -16,7 +16,7 @@ Tribute.mockImplementation(() => ({
 }))
 
 function mountNcRichContenteditable({ props, attrs }: any = {}) {
-	let currentValue = props?.modelVvalue
+	let currentValue = props?.modelVvalue ?? ''
 
 	const wrapper = mount(NcRichContenteditable, {
 		props: {