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 RxJsonSchema generic #1097

Merged
merged 1 commit into from
May 6, 2019
Merged

Conversation

phil-lgr
Copy link
Contributor

@phil-lgr phil-lgr commented May 2, 2019

This PR contains:

  • IMPROVED typings

Describe the problem you have without this PR

Make RxJsonSchema optionally generic to have type checking on properties and required.

Todos

  • Tests
  • Documentation
  • Typings
  • Changelog

@phil-lgr
Copy link
Contributor Author

phil-lgr commented May 2, 2019

I removed http-server from package.json because of jfhbrook/node-ecstatic#255

@phil-lgr
Copy link
Contributor Author

phil-lgr commented May 2, 2019

close #1093

@thornjad
Copy link

thornjad commented May 2, 2019

jfhbrook/node-ecstatic#256 will fix http-server

@phil-lgr phil-lgr force-pushed the generic-rx-json-schema branch from 1c643d8 to cae728f Compare May 3, 2019 19:26
@phil-lgr
Copy link
Contributor Author

phil-lgr commented May 3, 2019

I fixed my commit, I forgot to push something, waiting on CI and we should be good? is the v8.1.1 ok? @pubkey

Copy link
Owner

@pubkey pubkey left a comment

Choose a reason for hiding this comment

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

Thank you for the PR. The code looks good. I added two comments on what should be changed. Then we can merge.

CHANGELOG.md Outdated

Typings:
- `RxDocument.getAttachment()` and `RxDocument.allAttachments()` did not return promises

### 8.1.1 (2 May 2019)

Typings:
Copy link
Owner

Choose a reason for hiding this comment

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

Add your changes to the coming-soon block. Because I do not know atm when the next release will be and this would lead people into beleaving that 8.1.1 is already released.

package.json Outdated
@@ -1,7 +1,7 @@
{
"name": "rxdb",
"description": "A realtime Database for the Web",
"version": "8.1.0",
"version": "8.1.1",
Copy link
Owner

Choose a reason for hiding this comment

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

Do not change the version. This will happen on release.

UPDATE(docs) update RxJsonSchema w generic example
@phil-lgr phil-lgr force-pushed the generic-rx-json-schema branch from cae728f to 1584603 Compare May 6, 2019 14:16
@phil-lgr
Copy link
Contributor Author

phil-lgr commented May 6, 2019

Updated the version and README.md! Thanks!

@pubkey pubkey merged commit 8e842be into pubkey:master May 6, 2019
@pubkey
Copy link
Owner

pubkey commented May 6, 2019

Awesome. Thank you very much. Merged.

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.

3 participants