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
Fix part of #7176: Add types for Customization Args #9463
Conversation
Hi, @nishantwrp. This pull request does not have a "CHANGELOG: ..." label as mentioned in the PR checkbox list. Please add this label. PRs without this label will not be merged. If you are unsure of which label to add, please ask the reviewers for guidance. Thanks! |
Assigning @bansalnitish for the first-pass review of this pull request. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Just a few comments.
_computeBfsTraversalOfStates(initStateId: string, states: any, | ||
sourceStateName: string): Array<any> { | ||
|
||
_computeBfsTraversalOfStates(initStateId: string, states: States, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
break after (
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
computeBfsTraversalOfStates(initStateId: string, states: any, | ||
sourceStateName: any): Array<any> { | ||
|
||
computeBfsTraversalOfStates(initStateId: string, states: States, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
break after (
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
ICodeReplCustomizationArgs | IContinueCustomizationArgs | | ||
IDragAndDropSortInputCustomizationArgs | IEndExplorationCustomizationArgs | | ||
IFractionInputCustomizationArgs | IGraphInputCustomizationArgs | | ||
IImageClickInputCustomizationArgs | IInteractiveMapCustomizationArgs | | ||
IItemSelectionInputCustomizationArgs | ILogicCustomizationArgs | | ||
IMultipleChoiceInputCustomizationArgs | IMusicNotesInputCustomizationArgs | | ||
IPencilCodeCustomizationArgs | ISetInputCustomizationArgs | | ||
ITextInputCustomizationArgs); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
put every type on a separate line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@brianrodri, @kevintab95, @ankita240796, @aks681 and @marianazangrossi PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for code-owner files. Thanks @nishantwrp!
@marianazangrossi @brianrodri @aks681 PTAL! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm as codeowner
core/templates/components/state-editor/state-editor-properties-services/state-editor.service.ts
Outdated
Show resolved
Hide resolved
core/templates/components/state-editor/state-editor-properties-services/state-editor.service.ts
Outdated
Show resolved
Hide resolved
core/templates/components/state-editor/state-editor-properties-services/state-editor.service.ts
Outdated
Show resolved
Hide resolved
...ions/interactions/MathExpressionInput/directives/math-expression-input-validation.service.ts
Outdated
Show resolved
Hide resolved
...ions/interactions/MathExpressionInput/directives/math-expression-input-validation.service.ts
Outdated
Show resolved
Hide resolved
...ions/interactions/MultipleChoiceInput/directives/multiple-choice-input-validation.service.ts
Show resolved
Hide resolved
extensions/interactions/NumericInput/directives/numeric-input-validation.service.ts
Outdated
Show resolved
Hide resolved
extensions/interactions/NumericInput/directives/numeric-input-validation.service.ts
Outdated
Show resolved
Hide resolved
@marianazangrossi PTAL! |
@nithusha21 PTAL! |
@nithusha21 ptal! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as a codeowner!
* Add types for Customization Args * Copyright Notice * Reviews * lint tests * reviews * reviews * reviews
Overview
Essential Checklist
PR Pointers