Skip to content

🐛 changed default backend#585

Merged
danielmursa-dev merged 1 commit intomasterfrom
issue/change-default-filter-backend
May 22, 2025
Merged

🐛 changed default backend#585
danielmursa-dev merged 1 commit intomasterfrom
issue/change-default-filter-backend

Conversation

@TimdeBeer1
Copy link
Contributor

@TimdeBeer1 TimdeBeer1 commented May 14, 2025

Fixes maykinmedia/open-klant#414

Changes
Changed default filter backend and used commonground-api-common branch:
maykinmedia/commonground-api-common#94

@codecov-commenter
Copy link

codecov-commenter commented May 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.66%. Comparing base (f606ef9) to head (7cd61a8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #585   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files         148      148           
  Lines        5171     5171           
=======================================
  Hits         4895     4895           
  Misses        276      276           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator

@stevenbal stevenbal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When a new version of commonground-api-common is released, you can remove the git dependency from base.in and run bin/compile_dependencies.sh -P commonground-api-common==... to upgrade to that new version.

Also might be better to change the commit message to mention that you upgraded the library to version 2.x.x

@TimdeBeer1 TimdeBeer1 force-pushed the issue/change-default-filter-backend branch from 051d3e2 to b42f73b Compare May 21, 2025 07:21
@TimdeBeer1 TimdeBeer1 requested a review from stevenbal May 21, 2025 07:48
@TimdeBeer1 TimdeBeer1 force-pushed the issue/change-default-filter-backend branch from 3afbb3d to c2dd91f Compare May 21, 2025 12:51
@danielmursa-dev danielmursa-dev force-pushed the issue/change-default-filter-backend branch from c2dd91f to 4f0e9de Compare May 21, 2025 14:47
@danielmursa-dev danielmursa-dev force-pushed the issue/change-default-filter-backend branch from 4f0e9de to 7cd61a8 Compare May 21, 2025 14:49
@danielmursa-dev danielmursa-dev merged commit 8d9e92b into master May 22, 2025
22 checks passed
@danielmursa-dev danielmursa-dev deleted the issue/change-default-filter-backend branch May 22, 2025 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OnderwerpObject filters are missing help text in the API spec

4 participants