Skip to content

Modernization #407

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 61 commits into
base: master
Choose a base branch
from
Draft

Modernization #407

wants to merge 61 commits into from

Conversation

pokornyd
Copy link
Member

@pokornyd pokornyd commented Jul 22, 2025 •
edited
Loading

Motivation

Encompasses the modernization efforts on the delivery SDK, including but not limited to:

  • Refit adoption for declarative API implementation
  • ditching Newtonsoft.Json in favor of STJ
  • reworked, fluent-style querying logic
  • unified responses and improved error handling
  • reworked filtering
  • pluggable delegating handlers
  • structured rich text adoption

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

@pokornyd pokornyd self-assigned this Jul 24, 2025
@pokornyd pokornyd linked an issue Sep 9, 2025 that may be closed by this pull request
Proposal: New way of rendering rich-text #259
Open
@pokornyd pokornyd added rich-text Features and bugs related to rich-text resolution. filters Stories related to queries and filters. labels Sep 11, 2025
@pokornyd pokornyd modified the milestone: modernization Sep 11, 2025
@pokornyd pokornyd linked an issue Sep 11, 2025 that may be closed by this pull request
Surface deep HTTP errors #409
Open
@pokornyd
adjust folder structure
8373f0a
@pokornyd
register model resolution, add rt postprocessing
f193e64
@pokornyd
revamp generated models
a9d6573
@pokornyd
adjust used in queries and response
dbd191e
@pokornyd
slight refactor, unused cleanup
5b5917c
@pokornyd
adjust tests
245d30f
@pokornyd
lint & format
e0cb16b
@pokornyd
refactor RT, models and auth handler, fix related tests
9dc316c
@pokornyd
add filtering to enum query
3fc4974
@pokornyd
remove DI extensions and related tests
382f744
@pokornyd
fix response types for tax and types queries
1ab195b
@pokornyd
change response models to init only records
85cadd7
@pokornyd
refactor caching, add cached queries
8212591
@pokornyd
refactor reactive imple
22254d4
@pokornyd
fix failing tests, adjust test data
2273da1
@pokornyd
drop observable package and related tests
30c1718
@pokornyd
fix filtering queryparam generation
30d84a6
@pokornyd
make options mutable, refactor extensions, add asset element mapping
0814a63
@pokornyd
merge element conversion and deserialization
86a05ce
@pokornyd
refactor model provision, cleanup unused code
9bd3314
@pokornyd
unify element conversion and processing
1a36157
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pokornyd pokornyd

Labels
enhancement filters Stories related to queries and filters. rich-text Features and bugs related to rich-text resolution.
Projects
None yet
Milestone
modernization
1 participant
@pokornyd