-
Notifications
You must be signed in to change notification settings - Fork 73
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
[8.0.0-alpha] overview of options is not shown #3702
Comments
Have you verified, you cleared your cache? I cannot reproduce that behavior. |
Yes I cleared the browser caches.
|
Any relevant information inside the browser console? Something seems to prevent loading the app properly. |
I just tried another user account, for wich I never used polls before- and there it worked as expected. I just a thrid account, which already had polls before, showing the "old" polls or creating a new one - in both caeses, it does not work. In case of the error the browser console shows: TypeError: (intermediate value)(...).user is null Jürgen |
I just tried: adding another user as administrator of the poll. That other user sees the options, i.e. everthing is ok. But still the original user does not see the options. Any Idea? Jürgen |
Not until now. I am wondering where this comes from. But your observations are helpful, although I did not get behind the reason.
Maybe the network tab can reveal an information, which user is null. I think I have to build the alpha using dev mode, to get better information about the failing line. The linked code is not helpful, since it is minified. |
I found another strange thing, the following message is written to nextcloud.log, but only in the error cases (user with "old polls")
it comes from the PHP file
If I add some debug info that source the columnname causing that error ist
May be that "orphaned" user is the user where the javascript interpreter complains
(as shown in a previous messsage) If I use another nextcloud user, who is now administrator of that poll, no error happens, and that user sees everything fine as it should be. Any Idea about that? |
First of all thanks for the hint of
I shortened this names. But i doubt, that this is the cause, since it would then also cause an error with the privileged users. Seems that some information is cut of for non privileged users, but I could not reproduce it until now. Anyways, I changed some code also regarding the user properties. Additionally I changed the build to a dev build for alpha releases, so the console errors should lead to unminified code. Did you install the alpha via appstore or manually? |
I used the "standard" web-update mechanism. If you tell me where i get the "alpha-code", fpr testeing, I can douwnload it and install it. Jürgen |
Just wanted to know. You should find a new alpha6 in the appstore. I fixed some bugs and did some refactoring regarding users. Also the build is compiled as a dev build without minified code (I hope). |
alpha7 is out, too. BTW: If you do not find the release via the appstore, the latest builds are always found here. |
gli |
After the update, polls does not work anymore. That happens for all users. |
ok several deinstallation and installation of polls, polls shows up again, but the initial error ist still present. |
Ah, yeah. Damned. Since I changed the routes from the routes.php to attributes, there is no more named route for the main index. Have to fiddle that out. But you are still able to enter the app via the url.
|
Fixed in alpha8 |
I also can't reproduce that. But I still need information from the browser log, to be able to analyze that or access to the instance. Everything else is guessing. |
Thanks, but this are the server logs, which do not mention any relevant error (Besides some wrong named routes, which I fixed in #3749) What I think I need is information from the browser's console, like you quoted in #3702 (comment) |
Here the borwser console logs File 1: clicking on a poll, which showed the error console-export-2024-10-13_20-26-17.txt File 2: error when clicking the "Einstellungen" button |
Since I need polls urgently, I deleted the app an installed manually (unpacking the tgz.gz file) polls-v7.2.4. Jürgen |
Good to hear. But from your words, I suspect you are running a productive installation with the beta channel. I strongly suggest, you switch back to the stable channel, since betas - and especially alphas - are expected to be not production ready and contain bugs. Anyways, thanks for your effort and input. This is very valuable. |
What went wrong, what did you observe?
After update to polls 8.0.0. alpha the middle part of the polls view with all the details is not shown anymore:
If I view an exported link of the poll, everything is ok:
What did you expect, how polls should behave instead?
The details of the second screenshot should be displayed ion the "owenwers" view, as described in
hiere:
What steps does it need to replay this bug?
Installation method
Installed/updated from the appstore (Apps section of your site)
Installation type
Updated from a minor version within same major version (i.e. 4.0.0 to 4.1.1)
Affected polls version
8.0.0.0 alpha 4
Which browser did you use, when experiencing the bug?
Other browser
No response
Add your browser log here
No response
Additional client environment information
No response
NC version
Other/Don't know
Other Nextcloud version
No response
PHP engine version
PHP 8.2
Other PHP version
Nextcloud Hub 9 (30.0.0 RC4)
Database engine
MariaDB
Database Engine version or other Database
No response
Which user-backends are you using?
Add your nextcloud server log here
No response
Additional environment informations
No response
Configuration report
List of activated Apps
No response
Nextcloud Signing status
Additional Information
No response
The text was updated successfully, but these errors were encountered: