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

Add filter to the entries Export status #260

Merged
merged 4 commits into from
May 16, 2024

Conversation

pedro-mendonca
Copy link
Contributor

Description

This allows to set a custom set of statuses, like 'current_or_untranslated'.

Fix #259

How has this been tested?

  1. Set a different value on the status filter.
  2. Check out the language packs as desired.

Types of changes
New feature (non-breaking change which adds functionality)

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code has proper inline documentation.

swissspidy
swissspidy previously approved these changes Feb 2, 2024
@swissspidy swissspidy enabled auto-merge (squash) February 2, 2024 13:14
@ocean90 ocean90 added this to the 4.0.0 milestone May 16, 2024
inc/Export.php Outdated Show resolved Hide resolved
@swissspidy
Copy link
Collaborator

Once #270 is merged, this one should be good to go.

Copy link

codecov bot commented May 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.78%. Comparing base (8a4a120) to head (1f646e1).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #260      +/-   ##
============================================
+ Coverage     72.45%   79.78%   +7.32%     
  Complexity      350      350              
============================================
  Files            29       24       -5     
  Lines          1267     1014     -253     
============================================
- Hits            918      809     -109     
+ Misses          349      205     -144     
Flag Coverage Δ
feature ?
php 79.78% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@swissspidy swissspidy merged commit 5f49923 into wearerequired:master May 16, 2024
11 checks passed
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.

Allow to customize the status of the strings to Export
3 participants