Skip to content

Conversation

@js3110
Copy link
Collaborator

@js3110 js3110 commented Jan 15, 2026

Issue

Closes #898

Description

Exclusions handling updated so that it is included in settings file upload/download

Definition of Done

  • Exclusions handled in settings upload
  • all other features still available

How to test

Add exclusions and try updating settinsg. Check both settings download and full zip download versions.

Contributor checklist

  • Code passes lintr checks
  • Code passes all unit tests
  • New logic covered by unit tests
  • New logic is documented
  • App or package changes are reflected in NEWS
  • Package version is incremented

Notes to reviewer

Anything that the reviewer should know before tacking the pull request?

@js3110 js3110 marked this pull request as ready for review January 15, 2026 14:27
Copy link
Collaborator

@Shaakon35 Shaakon35 left a comment

Choose a reason for hiding this comment

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

Amazing!

…ata-upload' into 898-enhancement-add-general-exclusions-to-settings-upload-processing
Copy link
Collaborator

@Gero1999 Gero1999 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants