pixiebrix-extension
pixiebrix-extension copied to clipboard
#8761: update internal/external naming from `brick` to `package` in the Workshop
What does this PR do?
- Closes #8761
- Updates internal/external naming from brick to package. (Because not everything is a "brick" anymore)
Remaining Work
- [x] Fix broken playwright tests - having trouble with authentication fixture when running locally
Discussion
-
packageis a reserved keyword in JS. So uses a modifier, e.g.,editablePackage,packageMetadata, etc. where appropriate
Demo
Future Work
- Rename
BrickIcontoPackageIconbecause it's also used for mods/integrations - Rename
Metadatato something more informative of how it's actually used, e.g.,PackageBase: https://github.com/pixiebrix/pixiebrix-extension/blob/dd6e58e650d16acee0e8318cb2d4546b5bccb50a/src/types/registryTypes.ts#L63-L63
For more information on our expectations for the PR process, see the code review principles doc