Skip to content

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Jun 10, 2025

☑️ Resolves

  • Fix v-model usage in Vue 2 (for backward compatibility with main branch)
  • Noticed while testing unreleased eslint config

🖼️ Screenshots

🏚️ Before 🏡 After
Try stable8--nextcloud-vue Try deploy-preview

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 2️⃣ Backport to stable8 for maintained Vue 2 version or not applicable

@Antreesy Antreesy added this to the 8.27.1 milestone Jun 10, 2025
@Antreesy Antreesy requested review from DorraJaouad and susnux June 10, 2025 15:45
@Antreesy Antreesy self-assigned this Jun 10, 2025
@Antreesy Antreesy added bug Something isn't working 3. to review Waiting for reviews feature: select Related to the NcSelect* components labels Jun 10, 2025
@Antreesy Antreesy force-pushed the fix/stable8/nc-select-users branch from 01d7f69 to 75e6606 Compare June 10, 2025 15:50
@Antreesy Antreesy changed the title [stable8] fix(NcSelectUsers): fxi using of v-model in Vue 2 [stable8] fix(NcSelectUsers): fix using of v-model in Vue 2 Jun 10, 2025
Comment on lines 387 to 394
<script>
/** For backward compatibility with Vue 2 */
export default {
model: {
prop: 'modelValue',
event: 'update:modelValue',
},
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move this above template? script - template - script seems weird

@susnux susnux requested a review from ShGKme June 10, 2025 18:48
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the fix/stable8/nc-select-users branch from 75e6606 to fd50f51 Compare June 11, 2025 08:01
@Antreesy Antreesy enabled auto-merge June 11, 2025 08:01
@Antreesy Antreesy merged commit 74e5a6a into stable8 Jun 11, 2025
23 checks passed
@Antreesy Antreesy deleted the fix/stable8/nc-select-users branch June 11, 2025 08:04
@susnux susnux mentioned this pull request Jul 14, 2025
@susnux susnux modified the milestones: 8.27.1, 8.28.0 Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working feature: select Related to the NcSelect* components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants