Port the ReviewDialogView to TypeScript/spina.
Review Request #13157 — Created July 21, 2023 and submitted — Latest diff uploaded
This change ports the ReviewDialogView over to TypeScript. This is
pretty mechanical throughout. The only wart that this uncovered is that
theResourceCollectionOptions
needed to mark the fields as optional.
- Ran js-tests.
- Used the review dialog with different comment types.
Diff Revision 2
This is not the most recent revision of the diff. The latest diff is revision 3. See what's changed.
orig
1
2
3
Commits
Files
reviewboard/static/rb/js/common/resources/collections/resourceCollection.ts | ||
reviewboard/static/rb/js/reviews/index.ts | ||
reviewboard/static/rb/js/reviews/views/reviewDialogView.ts Was reviewboard/static/rb/js/views/reviewDialogView.es6.js | ||
reviewboard/static/rb/js/reviews/views/reviewRequestActions.ts | ||
reviewboard/static/rb/js/reviews/views/reviewablePageView.ts | ||
reviewboard/static/rb/js/reviews/views/tests/index.ts | ||
reviewboard/static/rb/js/reviews/views/tests/reviewDialogViewTests.ts Was reviewboard/static/rb/js/views/tests/reviewDialogViewTests.es6.js | ||
reviewboard/static/rb/js/reviews/views/tests/reviewablePageViewTests.ts |
reviewboard/static/rb/js/common/resources/collections/resourceCollection.ts |
---|
reviewboard/static/rb/js/reviews/index.ts |
---|
reviewboard/static/rb/js/views/reviewDialogView.es6.js |
---|
reviewboard/static/rb/js/reviews/views/reviewRequestActions.ts |
---|
reviewboard/static/rb/js/reviews/views/reviewablePageView.ts |
---|
reviewboard/static/rb/js/reviews/views/tests/index.ts |
---|
reviewboard/static/rb/js/views/tests/reviewDialogViewTests.es6.js |
---|
reviewboard/static/rb/js/reviews/views/tests/reviewablePageViewTests.ts |
---|