Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WellLogViewer, Typescript - Wrong type definitions #2259

Closed
Anders2303 opened this issue Sep 23, 2024 · 3 comments · Fixed by #2249
Closed

WellLogViewer, Typescript - Wrong type definitions #2259

Anders2303 opened this issue Sep 23, 2024 · 3 comments · Fixed by #2249
Labels
bug Something isn't working released well-log-viewer

Comments

@Anders2303
Copy link
Collaborator

Some of the exported types in the projects are incorrect, compared to what is actually valid for the options (atleast as far as I can see)

In WellLogTypes.ts, discrete objects are typed to an empty array, but it should be Record<string, any[]>

export type WellLogMetadataDiscreteObjects = Record<string, []>;

In WellLogTemplateTypes.ts, both color and type is required, but you'd want to omit them if you're using the style property

export type TemplatePlotProps = {
    type: TemplatePlotTypes;
    // ...
    color: CSSColor;
    // ...
};
@Anders2303 Anders2303 added bug Something isn't working well-log-viewer labels Sep 23, 2024
@w1nklr w1nklr closed this as completed in 96bcab4 Oct 3, 2024
hkfb pushed a commit that referenced this issue Oct 3, 2024
# [1.0.0](https://github.com/equinor/webviz-subsurface-components/compare/wsc-common@0.8.9...wsc-common@1.0.0) (2024-10-03)

### Features

* ColorMapFunction type in WellLogView ([#2249](#2249)) ([96bcab4](96bcab4)), closes [#2054](#2054) [#2259](#2259)

### BREAKING CHANGES

* - colorMapFunctions array replaces colorTables array to combine color
tables and functions in single array
- colorMapFunctionName and reverseColorMapFunctionName references
replace colorTable and reverseColorTable references (JSON files also
should be changed)
@hkfb
Copy link
Collaborator

hkfb commented Oct 3, 2024

🎉 This issue has been resolved in version wsc-common@1.0.0 🎉

The release is available on GitHub release

@hkfb hkfb added the released label Oct 3, 2024
hkfb pushed a commit that referenced this issue Oct 3, 2024
# [1.0.0](https://github.com/equinor/webviz-subsurface-components/compare/subsurface-viewer@0.30.17...subsurface-viewer@1.0.0) (2024-10-03)

### Features

* ColorMapFunction type in WellLogView ([#2249](#2249)) ([96bcab4](96bcab4)), closes [#2054](#2054) [#2259](#2259)

### BREAKING CHANGES

* - colorMapFunctions array replaces colorTables array to combine color
tables and functions in single array
- colorMapFunctionName and reverseColorMapFunctionName references
replace colorTable and reverseColorTable references (JSON files also
should be changed)
@hkfb
Copy link
Collaborator

hkfb commented Oct 3, 2024

🎉 This issue has been resolved in version subsurface-viewer@1.0.0 🎉

The release is available on GitHub release

hkfb pushed a commit that referenced this issue Oct 3, 2024
# [2.0.0](https://github.com/equinor/webviz-subsurface-components/compare/well-log-viewer@1.13.11...well-log-viewer@2.0.0) (2024-10-03)

### Features

* ColorMapFunction type in WellLogView ([#2249](#2249)) ([96bcab4](96bcab4)), closes [#2054](#2054) [#2259](#2259)

### BREAKING CHANGES

* - colorMapFunctions array replaces colorTables array to combine color
tables and functions in single array
- colorMapFunctionName and reverseColorMapFunctionName references
replace colorTable and reverseColorTable references (JSON files also
should be changed)
@hkfb
Copy link
Collaborator

hkfb commented Oct 3, 2024

🎉 This issue has been resolved in version well-log-viewer@2.0.0 🎉

The release is available on GitHub release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released well-log-viewer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants