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

fix: list polls after taking over #3921

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

hamza221
Copy link
Contributor

fix #3920

@dartcafe
Copy link
Collaborator

@hamza221 Thanks for taking care, but the reported error is caused in 7.4.1, which is managed in the master-7 branch.

I am currently working on the errors regarding the poll lists. There are some more flaws.

@hamza221
Copy link
Contributor Author

hamza221 commented Mar 21, 2025

h is managed in the master-7 branch.

I am currently working on the errors regarding the poll lists. There are some more flaws.

Sorry I got confused with the branch names, makes more sense now. Should I open a new Pr against master7? and this should stay right, as It's a bug here too

@dartcafe
Copy link
Collaborator

h is managed in the master-7 branch.
I am currently working on the errors regarding the poll lists. There are some more flaws.

Sorry I got confused with the branch names, makes more sense now. Should I open a new Pr against master7?

yeah. makes sense, since I don't know when 8.x will go into the next stage. Still some things to do and waiting for nextcloud-vue gets in stable state.

and this should stay right, as It's a bug here too

I prefer to solve it another way. Is on the to do list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin taking over poll does not update conter in app navigation
2 participants