Skip to content

Conversation

@stas-panasiuk
Copy link
Collaborator

No description provided.

@stas-panasiuk stas-panasiuk added checked on a cluster The code has been tested on a cluster. ready to review PR is ready to review labels Jun 2, 2023
@carolynduby carolynduby added the next release High priority Pull Requests that should be merged first label Jun 13, 2023

ngOnInit(): void {
this.store.dispatch(SampleFolderViewInitializedAction());
this.sampleFolderPath$.subscribe(value => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this call would probably be better moved to the constructor, since it initializes the folderForm field.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it won't work in that case, we need for the page to initialize before

# Conflicts:
#	flink-cyber/metron-parser-chain/parser-chains-config-service/src/main/java/com/cloudera/parserchains/queryservice/config/AppProperties.java
#	flink-cyber/metron-parser-chain/parser-chains-config-service/src/main/java/com/cloudera/parserchains/queryservice/controller/ChainController.java
@carolynduby carolynduby merged commit 6510a1f into develop Jul 13, 2023
@carolynduby carolynduby deleted the CYB-150 branch July 13, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

checked on a cluster The code has been tested on a cluster. next release High priority Pull Requests that should be merged first ready to review PR is ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants