Title: fix(database): allow null values for equalTo, etc. by cartant · Pull Request #809 · angular/angularfire · GitHub
Open Graph Title: fix(database): allow null values for equalTo, etc. by cartant · Pull Request #809 · angular/angularfire
X Title: fix(database): allow null values for equalTo, etc. by cartant · Pull Request #809 · angular/angularfire
Description: Checklist Issue number for this PR: #704 and #715 Docs included?: no Test units included?: yes e2e tests included?: no In a clean directory, npm install, npm run build, and npm test run successfully? yes Description The previous PR for #704 did not include tests and did not completely fix the issue, as null values were swallowed with the query observable. This PR addresses that and includes tests. Also, this PR ensures that the query observable emits undefined if values are not specified for equalTo, startAt or endAt - as null has a specific meaning.
Open Graph Description: Checklist Issue number for this PR: #704 and #715 Docs included?: no Test units included?: yes e2e tests included?: no In a clean directory, npm install, npm run build, and npm test run successful...
X Description: Checklist Issue number for this PR: #704 and #715 Docs included?: no Test units included?: yes e2e tests included?: no In a clean directory, npm install, npm run build, and npm test run successful...
Opengraph URL: https://github.com/angular/angularfire/pull/809
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:9f5eae96-03f0-df22-5b8e-bab3103e601c |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E91A:1CA2B7:5CD83E:79B2E3:6A5AE215 |
| html-safe-nonce | 0515812cdb89b757404d9d424fca10caf283f2fb71f0f7dd8ee8fc2453e3f98c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFOTFBOjFDQTJCNzo1Q0Q4M0U6NzlCMkUzOjZBNUFFMjE1IiwidmlzaXRvcl9pZCI6Ijg5NzE4MTEzNzE3MDYxNDczNDkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 93f2615bd54ffb61cdc5c52a8c92ff5de0304d42584cf594a8a6aa6ed0789c64 |
| hovercard-subject-tag | pull_request:104685440 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/angular/angularfire/pull/809/files |
| twitter:image | https://avatars.githubusercontent.com/u/3878593?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/3878593?s=400&v=4 |
| og:image:alt | Checklist Issue number for this PR: #704 and #715 Docs included?: no Test units included?: yes e2e tests included?: no In a clean directory, npm install, npm run build, and npm test run successful... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5290d7e14309ad1e76106a9c4237bd1041517e83ea182c8ab756752cb0c6940b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/angular/angularfire git https://github.com/angular/angularfire.git |
| octolytics-dimension-user_id | 139426 |
| octolytics-dimension-user_login | angular |
| octolytics-dimension-repository_id | 49453413 |
| octolytics-dimension-repository_nwo | angular/angularfire |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 49453413 |
| octolytics-dimension-repository_network_root_nwo | angular/angularfire |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 9c975978430e9ad293956f2bbdaf153b1bd84a99 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width