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 typescript 4.3 support #14

Merged
merged 4 commits into from
Aug 26, 2021
Merged

Add typescript 4.3 support #14

merged 4 commits into from
Aug 26, 2021

Conversation

konowrockis
Copy link
Contributor

The main goal of this pull request is to update TypeScript to 4.3 because ts-routes 1.0 is not compatible with TS > 4.0.

Side effect of that is that it addresses #2 and #9.

I migrated underlying implementation of query strings parsing and stringifying to qs because query-string does not support objects serialization. Also added benefit is that I've exposed configration for qs.

The types has simplified a bit and also I think it'll be easier to add new features in the future.

@konowrockis konowrockis added the enhancement New feature or request label Jun 18, 2021
@konowrockis konowrockis self-assigned this Jun 18, 2021
@konowrockis konowrockis force-pushed the feature/routes-2.0 branch 2 times, most recently from 8962691 to 0df3006 Compare June 21, 2021 07:10
README.md Show resolved Hide resolved
__tests__/segments/arg.test.ts Outdated Show resolved Hide resolved
src/query.ts Outdated Show resolved Hide resolved
@konowrockis konowrockis force-pushed the feature/routes-2.0 branch 2 times, most recently from 99fa3f6 to 8feb706 Compare July 5, 2021 09:28
README.md Outdated Show resolved Hide resolved
@konowrockis konowrockis merged commit f25bcf9 into master Aug 26, 2021
@konowrockis konowrockis deleted the feature/routes-2.0 branch August 26, 2021 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants