-
Notifications
You must be signed in to change notification settings - Fork 4
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
chore(Authoring): Convert project authoring to Angular #1371
Conversation
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.
Functionality seems to work as before. I have some comments:
- import-step/choose-location page (used when choosing the location for an imported step, automated assessment or interactive simulation) seems to have lost the indentation for the steps?
-
Add private/protected modifiers for ProjectAuthoringComponent class variables?
-
See inline comments
private ConfigService: ConfigService, | ||
private CopyNodesService: CopyNodesService, | ||
private DeleteNodeService: DeleteNodeService, | ||
private MoveNodesService: MoveNodesService, | ||
private ProjectService: TeacherProjectService, | ||
private TeacherDataService: TeacherDataService | ||
) { | ||
this.$translate = $filter('translate'); | ||
} | ||
|
||
$onInit() { | ||
this.items = this.ProjectService.idToOrder; | ||
private TeacherDataService: TeacherDataService, | ||
private upgrade: UpgradeModule |
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.
Lower case variable names, change TeacherDataServicer to dataService?
🎉 This PR is included in version 5.107.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Changes
Converted project authoring to Angular.
Test
Make sure everything in the project authoring still works including but not limited to
Closes #1366