-
-
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
(bug) sort playlist not working as expected #411
Comments
Hi! Make sure to use the version of nodejs specified in the .nvmrc file +
to “npm install” the dependencies after cloning/pulling the latest version
of the master branch. Let me know how it goes, ok?
Adrien
|
Otherwise, this could be caused by a caching issue coming from Docker, making it skip the "npm install" step. (As DD-trace is from the Datadog module we added recently) Can you make sure that you use the |
Alright I got it working 🎉 I did the following: 1 - Install node I will work on the solution for this and do a pull request later today. |
Awesome, thank you for sharing how you proceeded, Compiuta! 🤘 |
For reference, here's what my playlists page looks like today: I don't know if this is related to the fix you want to make, but as you can see, there are some holes (or blanks) in the list. |
@adrienjoly Is this happening after you use the sort functionality? I am not seeing the broken layout |
Nope, without using the sort functionality. I'm getting this issue on Brave and Firefox. EDIT: Actually, as soon as I pick a sort, the blanks disappear! 😯 |
I tested in firefox and chrome but I still do not get the broken layout. Do you also get this bug while browsing other users playlists or when you are logged out ? |
Closes #411 ## What does this PR do / solve? Currently the sort playlist feature is using the index and paylistname data attributes to sort by date and alphabet. The issue is that I did not realize that the array sort function takes into consideration lower and upper case when sorting. Also the index is being sorted as a string rather than a number which causes the playlist to be in the incorrect order when sorting by date. ## Overview of changes The sort function will now lower case all playlist titles and convert index strings into numbers before sorting to prevent these bugs.
I noticed that the sort by date option for playlists does not work as expected while a user has more than 10 playlists.
@adrienjoly I have the solution but there seems to be an issue with the dev environment right now.
After pulling all new updates from the master repository I am now getting errors and I am not able to connect to the localhost server. I also tried cloning a new copy of the master repository and I still get this error
Let me know if you are also having issues
The text was updated successfully, but these errors were encountered: