Skip to content

Update UrlExportSettings.tsx#336

Merged
lukasoppermann merged 2 commits intolukasoppermann:mainfrom
IddanCF:patch-1
Feb 16, 2026
Merged

Update UrlExportSettings.tsx#336
lukasoppermann merged 2 commits intolukasoppermann:mainfrom
IddanCF:patch-1

Conversation

@IddanCF
Copy link
Contributor

@IddanCF IddanCF commented Nov 28, 2025

Add missing option to set excludeExtensionProp in UrlExportSettings.
Solves #335.

@coveralls
Copy link

coveralls commented Nov 28, 2025

Pull Request Test Coverage Report for Build 19824605659

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 77.921%

Totals Coverage Status
Change from base Build 18306258487: 0.0%
Covered Lines: 655
Relevant Lines: 822

💛 - Coveralls

@0m4r
Copy link
Collaborator

0m4r commented Nov 28, 2025

@IddanCF can you please build the code and add the build files too?

@IddanCF
Copy link
Contributor Author

IddanCF commented Dec 1, 2025

@0m4r done! Please check again

Copy link
Collaborator

@0m4r 0m4r left a comment

Choose a reason for hiding this comment

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

it looks good to me.
@lukasoppermann to provide final approval ;)

@0m4r
Copy link
Collaborator

0m4r commented Dec 12, 2025

@IddanCF a previous PR has been merged.
Would you please take care of the merge conflicts?

@IddanCF
Copy link
Contributor Author

IddanCF commented Jan 26, 2026

@0m4r is there anything else needed from my side?

@0m4r
Copy link
Collaborator

0m4r commented Jan 26, 2026

@0m4r is there anything else needed from my side?

I think you are good now.
@lukasoppermann for the final approval and merge :)

@IddanCF
Copy link
Contributor Author

IddanCF commented Feb 13, 2026

Hey @0m4r @lukasoppermann,
Any updates on this?

Our workflow is blocked with message: "client_payload is too large.". Excluding the unused extension props will solve this for us. 🙏

@lukasoppermann lukasoppermann merged commit 7bb29ca into lukasoppermann:main Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants