Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: workflowengine embeds different Vue instances, but they might be incompatible #50114

Open
4 of 8 tasks
blizzz opened this issue Jan 9, 2025 · 5 comments
Open
4 of 8 tasks
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 31-feedback bug feature: workflows

Comments

@blizzz
Copy link
Member

blizzz commented Jan 9, 2025

⚠️ This issue respects the following points: ⚠️

Bug description

See nextcloud-libraries/nextcloud-vue#6360 for more information

Steps to reproduce

  1. see above

Expected behavior

apps hooking into the workflowengine have a working config interface independent of their used Vue version.

Nextcloud Server version

master

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

List of activated Apps

Nextcloud Signing status

Nextcloud Logs

Additional info

No response

@blizzz blizzz added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: workflows labels Jan 9, 2025
@blizzz
Copy link
Member Author

blizzz commented Jan 9, 2025

cc @juliusknorr

@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Jan 9, 2025
@blizzz blizzz moved this from 🧭 Planning evaluation (don't pick) to 📄 To do (~10 entries) in 📝 Office team Jan 9, 2025
@nickvergessen
Copy link
Member

This is also blocking Files automated tagging release nextcloud/files_automatedtagging#1076

@juliusknorr
Copy link
Member

I shared a few ideas of what to try with @blizzz but also just patterns that have worked to work around that in the past. The javascript API probably needs some redesign like a callback to just register a render function, but might be more work.

@nickvergessen
Copy link
Member

So downgrading to @nextcloud/vue@8.19.0 is the best option for now, and then for 32 we need to make a plan?

@blizzz
Copy link
Member Author

blizzz commented Jan 10, 2025

So downgrading to @nextcloud/vue@8.19.0 is the best option for now, and then for 32 we need to make a plan?

I have some things to try out, if they prove fine, there will be some documentation for app devs. Otherwise this is the workaround.

To solve this properly changes on the engine are necessary, and this is 99% backwards incompatible. Being already after feature freeze, this is rather something for 32.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 31-feedback bug feature: workflows
Projects
Status: 📄 To do (~10 entries)
Development

No branches or pull requests

4 participants