Skip to content

Commit

Permalink
Remove plural on devices engine collection property to avoid breaking…
Browse files Browse the repository at this point in the history
… change
  • Loading branch information
alexandrebouthinon committed Jun 6, 2024
1 parent 83c2ff7 commit e5837cf
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion doc/2/concepts/configuration/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ The following collections are used by the plugin. They're automatically created
The following collections are used by the plugin. They're automatically created when the plugin is started and are specific to each engine:

- `config`: The collection that stores the engine configuration.
- `devices`: The collection that stores the devices.
- `device`: The collection that stores the devices.
- `measures`: The collection that stores the measures once extracted by the decoders.
- `asset`: The collection that stores the assets.
- `assetGroups`: The collection that stores the assets groups.
Expand Down
2 changes: 1 addition & 1 deletion lib/modules/plugin/DeviceManagerPlugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,7 @@ export class DeviceManagerPlugin extends Plugin {
assetHistory: {
name: InternalCollection.ASSETS_HISTORY,
},
devices: {
device: {
name: InternalCollection.DEVICES,
mappings: devicesMappings,
},
Expand Down
4 changes: 3 additions & 1 deletion lib/modules/plugin/types/DeviceManagerConfiguration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ export type DeviceManagerConfiguration = {
mappings: JSONObject;
settings?: JSONObject;
};
// ! TODO next major release: pluralize the collection name
asset: {
name: string;
mappings: CollectionMappings;
Expand All @@ -65,7 +66,8 @@ export type DeviceManagerConfiguration = {
name: string;
settings?: JSONObject;
};
devices: {
// ! TODO next major release: pluralize the collection name
device: {
name: string;
mappings: CollectionMappings;
settings?: JSONObject;
Expand Down

0 comments on commit e5837cf

Please sign in to comment.