Skip to content

Commit

Permalink
change on this.props.isDefault
Browse files Browse the repository at this point in the history
Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>
  • Loading branch information
zhyuanqi committed Mar 19, 2024
1 parent 813d4f3 commit 4d0bb20
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- [Chrome] Introduce registerCollapsibleNavHeader to allow plugins to customize the rendering of nav menu header ([#5244](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5244))
- [Dynamic Configurations] Pass request headers when making application config calls ([#6164](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6164))
- [Discover] Options button to configure legacy mode and remove the top navigation option ([#6170](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6170))
- [Multiple Datasource] Add default functionality for customer to choose default datasource ([#6058](https://github.com/opensearch-project/OpenSearch-Dashboards/issues/6058))


### 🐛 Bug Fixes
Expand Down Expand Up @@ -141,7 +142,6 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- [Discover] Enhanced the data source selector with added sorting functionality ([#5719](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5719))
- [Multiple Datasource] Add datasource picker component and use it in devtools and tutorial page when multiple datasource is enabled ([#5756](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5756))
- [Multiple Datasource] Add datasource picker to import saved object flyout when multiple data source is enabled ([#5781](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5781))
- [Multiple Datasource] Add default functionality for customer to choose default datasource ([#6058](https://github.com/opensearch-project/OpenSearch-Dashboards/issues/6058))

### 🐛 Bug Fixes

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -408,10 +408,6 @@ export class EditDataSourceForm extends React.Component<EditDataSourceProps, Edi
}
};

isDefault = () => {
return this.props.isDefault;
};

onClickTestConnection = async () => {
this.setState({ isLoading: true });
const isNewCredential = !!(this.state.auth.type !== this.props.existingDataSource.auth.type);
Expand Down Expand Up @@ -647,7 +643,7 @@ export class EditDataSourceForm extends React.Component<EditDataSourceProps, Edi
dataSourceName={this.props.existingDataSource.title}
onClickTestConnection={this.onClickTestConnection}
onClickSetDefault={this.setDefaultDataSource}
isDefault={this.isDefault()}
isDefault={this.props.isDefault}
/>
);
};
Expand Down

0 comments on commit 4d0bb20

Please sign in to comment.