parse-community / parse-dashboard

A dashboard for managing Parse Server
https://parseplatform.org
Other
3.74k stars 1.38k forks source link

fix: Crash after setting filter using context menu and opening filters panel #2579

Closed 404-html closed 1 month ago

404-html commented 1 month ago

New Pull Request Checklist

Issue Description

Fixes: https://github.com/parse-community/parse-dashboard/issues/2578

Approach

Seems that filter now have additional property: class. When setting filter using context menu this property was not provided. This was causing an issue when binding filters. I did following:

  1. Setting 'class' on filter when using context menu filters
  2. In addition, when parsing filters from query strings, setting class if it's missing; this should provide some backward compatibility

TODOs before merging

parse-github-assistant[bot] commented 1 month ago

Thanks for opening this pull request!

uffizzi-cloud[bot] commented 1 month ago

Uffizzi Ephemeral Environment deployment-54094

:watch: Updated Jul 11, 2024, 12:36 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2579

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

parse-github-assistant[bot] commented 1 month ago

I will reformat the title to use the proper commit message syntax.

mtrezza commented 1 month ago

Nice, especially the backward compatibility.

parseplatformorg commented 1 month ago

🎉 This change has been released in version 6.0.0-alpha.10