From 1c50065e6357d88133b71c7b730f1c247f61996c Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Mon, 25 Nov 2024 11:50:37 +0100
Subject: [PATCH 1/2] fix(NcActionRadio): change modelValue to behave like
 NcCheckboxRadioSwitch

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../NcActionRadio/NcActionRadio.vue           | 47 ++++++++++++-------
 1 file changed, 29 insertions(+), 18 deletions(-)

diff --git a/src/components/NcActionRadio/NcActionRadio.vue b/src/components/NcActionRadio/NcActionRadio.vue
index 03b0d273b1..1bb4dc97b4 100644
--- a/src/components/NcActionRadio/NcActionRadio.vue
+++ b/src/components/NcActionRadio/NcActionRadio.vue
@@ -10,27 +10,34 @@ So that only one of each name set can be selected at the same time.
 
 ```vue
 <template>
-	<NcActions>
-		<NcActionRadio @change="alert('(un)checked !')" name="uniqueId">First choice</NcActionRadio>
-		<NcActionRadio value="second" v-model="radioValue" name="uniqueId" @change="alert('(un)checked !')">Second choice (v-model)</NcActionRadio>
-		<NcActionRadio :model-value="true" name="uniqueId" @change="alert('(un)checked !')">Third choice (checked)</NcActionRadio>
-		<NcActionRadio :disabled="true" name="uniqueId" @change="alert('(un)checked !')">Fourth choice (disabled)</NcActionRadio>
-	</NcActions>
+	<div>
+		<NcActions>
+			<NcActionRadio v-for="option in radioOptions"
+				:key="option.value"
+				:value="option.value"
+				:disabled="option.disabled"
+				name="uniqueId"
+				v-model="radioValue">
+				{{ option.label }}
+			</NcActionRadio>
+		</NcActions>
+		<span>Selected value: {{ radioValue }}</span>
+	</div>
 </template>
 
 <script>
 	export default {
 		data() {
 			return {
-				radioValue: false,
+				radioOptions: [
+					{ value: 'first', label: 'First choise', disabled: false },
+					{ value: 'second', label: 'Second choise', disabled: false },
+					{ value: 'third', label: 'Third choise', disabled: false },
+					{ value: 'fourth', label: 'Fourth choise (disabled)', disabled: true },
+				],
+				radioValue: 'first',
 			}
 		},
-
-		methods: {
-			alert(message) {
-				alert(message)
-			}
-		}
 	}
 </script>
 ```
@@ -41,8 +48,8 @@ So that only one of each name set can be selected at the same time.
 		<span class="action-radio" role="menuitemradio" :aria-checked="ariaChecked">
 			<input :id="id"
 				ref="radio"
+				v-model="model"
 				:disabled="disabled"
-				:checked="checked"
 				:name="name"
 				:value="value"
 				:class="{ focusable: isFocusable }"
@@ -100,11 +107,11 @@ export default {
 		},
 
 		/**
-		 * checked state of the the radio element
+		 * checked state of the radio element
 		 */
 		modelValue: {
-			type: Boolean,
-			default: false,
+			type: [String, Number],
+			default: '',
 		},
 
 		/**
@@ -186,7 +193,11 @@ export default {
 			this.$refs.label.click()
 		},
 		onChange(event) {
-			this.model = this.$refs.radio.checked
+			if (this.checked !== undefined) {
+				this.model = this.$refs.radio.checked
+			} else {
+				this.model = this.value
+			}
 
 			/**
 			 * Emitted when the radio state is changed

From 938ef32cd6abed7da333612cfcdd4b2eabd21116 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Mon, 27 Jan 2025 13:19:54 +0100
Subject: [PATCH 2/2] chore: add hint to CHANGELOG.md

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 CHANGELOG.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e27ab7f3f6..68c8d0ffbc 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,8 @@
   - to `import NcButton from '@nextcloud/vue/components/NcButton'`
 
   The old import paths are still valid, but deprecated and will be removed in version 9.
+* `NcActionRadio` is now expecting String|Number in `v-model` directive (to compare with passed `value`) instead of Boolean. Consider it for migration.
+
 
 ## [v8.22.0](https://github.com/nextcloud-libraries/nextcloud-vue/tree/v8.22.0) (2024-12-20)
 [Full Changelog](https://github.com/nextcloud-libraries/nextcloud-vue/compare/v8.21.0...v8.22.0)