[COMPASS-7124] Update `mongodb-schema` to work in browser environment without poly fills Created: 17/Aug/23 Updated: 06/Nov/23 Resolved: 30/Oct/23 |
|
| Status: | Closed |
| Project: | Compass |
| Component/s: | Schema, Tech debt |
| Affects Version/s: | None |
| Fix Version/s: | 1.41.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Rhys Howell | Assignee: | Anna Henningsen |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Epic Link: | COMPASS-7020 |
| Story Points: | 3 |
| Documentation Changes: | Not Needed |
| Sprint: | Iteration Minmi, Iteration Nodosaurus |
| Description |
|
In recent changes: https://github.com/mongodb-js/mongodb-schema/pull/183 This ticket involves updating the stream usage to one that doesn't need polyfills. Let's also add a browser based test so this can't regress in future from other packages. |
| Comments |
| Comment by Githook User [ 06/Nov/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}Message: chore(deps): bump mongodb-schema to 12.0.0 |
| Comment by Githook User [ 01/Nov/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}Message: chore(deps): bump mongodb-schema to 12.0.0 |
| Comment by Githook User [ 31/Oct/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}Message: chore(deps): bump mongodb-schema to 12.0.0 |
| Comment by Githook User [ 30/Oct/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}Message: chore(deps): bump mongodb-schema to 12.0.0 |
| Comment by Githook User [ 30/Oct/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}Message: chore(deps): bump mongodb-schema to 12.0.0 |
| Comment by Githook User [ 30/Oct/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}Message: feat!: drop streams support We don't really need streams usage anymore and can just drop it. That's |
| Comment by Githook User [ 30/Oct/23 ] |
|
Author: {'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}Message: feat!: drop streams support We don't really need streams usage anymore and can just drop it. That's |
| Comment by Anna Henningsen [ 18/Aug/23 ] |
|
The comment that triggered this suggested using stream-browserify, but just a note, that package is only a wrapper around readable-stream with legacy handling. We shouldn’t need any legacy streams handling here and can just use readable-stream on its own. |