chore(samples): Flushing out samples (#2933)

* Splitting out the samples more
* Add sample callouts
* Small README changes

Author: jamesdaniels

.gitattributes

@@ -1,5 +1,5 @@
 src/**/firebase.ts linguist-generated=true
 src/**/rxfire.ts linguist-generated=true
 src/compat/**/base.ts linguist-generated=true
-sample/**/* linguist-generated=true
-sample-compat/**/* linguist-generated=true
+samples/**/* linguist-generated=true
+yarn.lock linguist-generated=true

README.md

@@ -87,8 +87,7 @@ Neither AngularFire or Firebase ship with polyfills. To have compatability acros
 | Various ES5+ features  | IE 11<br>Safari &lt; 10 | [`core-js/stable`](https://github.com/zloirock/core-js#readme) | MIT |
 | `globalThis` | [IE 11<br>Chrome &lt; 71<br>Safari &lt; 12.1<br>iOS &lt; 12.2<br>Node &lt; 12](https://caniuse.com/mdn-javascript_builtins_globalthis) | [`globalThis`](https://github.com/es-shims/globalThis#readme) | MIT |
 | `Proxy` | [IE 11<br>Safari &lt; 10](https://caniuse.com/proxy) | [`proxy-polyfill`](https://github.com/GoogleChrome/proxy-polyfill#readme) | Apache 2.0 |
-| `fetch` | [IE 11<br>Node<br>Safari &lt; 10.1<br>iOS &lt; 10.3](https://caniuse.com/fetch) | [`cross-fetch`](https://github.com/lquixada/cross-fetch#readme) | MIT |
-| `Headers` | [IE 11<br>Node<br>Safari &lt; 10.1<br>iOS Safari](https://caniuse.com/mdn-api_headers) | [`cross-fetch`](https://github.com/lquixada/cross-fetch#readme) | MIT |
+| `fetch` | [IE 11<br>Safari &lt; 10.1<br>iOS &lt; 10.3](https://caniuse.com/fetch) | [`cross-fetch`](https://github.com/lquixada/cross-fetch#readme) | MIT |
 
 ## Resources
 
@@ -100,7 +99,17 @@ Neither AngularFire or Firebase ship with polyfills. To have compatability acros
 
 [Upgrading to v7.0? Check out our guide.](docs/version-7-upgrade.md)
 
-**Having troubles?** Get help on our [Q&A board](https://github.com/angular/angularfire/discussions?discussions_q=category%3AQ%26A), the official [Firebase Mailing List](https://groups.google.com/forum/#!forum/firebase-talk), the [Firebase Community Slack](https://firebase.community/) (`#angularfire2`), the [Angular Community Discord](http://discord.gg/angular) (`#firebase`), [Gitter](https://gitter.im/angular/angularfire2), the [Firebase subreddit](https://www.reddit.com/r/firebase), or [Stack Overflow](https://stackoverflow.com/questions/tagged/angularfire2).
+### Sample apps
+
+We have three sample apps in this repository:
+
+1. [`samples/compat`](samples/compat) a kitchen sink application that demonstrates use of the "compatability" API in an Angular Universal app w/TransferState.
+1. [`samples/modular`](samples/modular) a kitchen sink application that demonstrates the new tree-shakable API in an Angular Universal app w/TransferState.
+1. [`samples/advanced`](samples/advanced) the same app as `samples/modular` but demonstrates more advanced concepts such as dynamically importing Firebase feature modules and Firestore data bundling.
+
+### Having troubles?
+
+Get help on our [Q&A board](https://github.com/angular/angularfire/discussions?discussions_q=category%3AQ%26A), the official [Firebase Mailing List](https://groups.google.com/forum/#!forum/firebase-talk), the [Firebase Community Slack](https://firebase.community/) (`#angularfire2`), the [Angular Community Discord](http://discord.gg/angular) (`#firebase`), [Gitter](https://gitter.im/angular/angularfire2), the [Firebase subreddit](https://www.reddit.com/r/firebase), or [Stack Overflow](https://stackoverflow.com/questions/tagged/angularfire2).
 
 > **NOTE:** AngularFire is maintained by Googlers but is not a supported Firebase product. Questions on the mailing list and issues filed here are answered on a <strong>best-effort basis</strong> by maintainers and other community members. If you are able to reproduce a problem with Firebase <em>outside of AngularFire's implementation</em>, please [file an issue on the Firebase JS SDK](https://github.com/firebase/firebase-js-sdk/issues) or reach out to the personalized [Firebase support channel](https://firebase.google.com/support/).
 

SECURITY.md

@@ -4,8 +4,8 @@
 
 | Version | Supported          |
 | ------- | ------------------ |
-| 5.0+    | :white_check_mark: |
-| < 5.0   | :x:                |
+| 6.0+    | :white_check_mark: |
+| < 6.0   | :x:                |
 
 ## Reporting a Vulnerability
 

docs/version-7-upgrade.md

@@ -164,7 +164,7 @@ collection<T>(firestore, 'foo') // CollectionReference<T>
 
 ```ts
 import { collectionGroup } from '@angular/fire/firestore';
-collectionGroup<T>('foo') // Query<T>
+collectionGroup<T>(firestore, 'foo') // Query<T>
 ```
 </td>
         </tr>

package.json

@@ -68,14 +68,8 @@
     "semver": "^7.1.3",
     "tslib": "^2.1.0",
     "webpack": "^5.35.0",
-    "ws": "^7.2.1",
-    "xhr2": "^0.1.4",
     "zone.js": "~0.11.4"
   },
-  "optionalDependencies": {
-    "bufferutil": "^4.0.1",
-    "utf-8-validate": "^5.0.2"
-  },
   "devDependencies": {
     "@angular-devkit/build-angular": "^12.0.0",
     "@angular/animations": "~12.0.0",