Summary

Update the API token policy editor from $.modalBox to Ink.DialogView.

Review Request #14162 — Created Sept. 12, 2024 and updated Sept. 16, 2024, 8:20 a.m.

Information

Owner

david

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

This change ports the API token policy editor pop-up from our old

modalBox function to the new DialogView.

Testing Done


Ran js-tests.
Verified that the functionality and appearance of the editor dialog worked as

  expected.

Commits

Summary	ID
Update the API token policy editor from $.modalBox to Ink.DialogView. This change ports the API token policy editor pop-up from our old `modalBox` function to the new `DialogView`. Testing Done: - Ran js-tests. - Verified that the functionality and appearance of the editor dialog worked as expected.	942b462c41ffa2f8a2cc3d7b2439c486556ccd0c

Issues

Description	From	Last Updated
Thoughts on using this format instead? super.initialize(_.defaults(options, { size: DialogSize.LARGE, })); I feel like I've seen us pass parent options …	maubin	Sept. 13, 2024, 9:01 a.m.

flake8 passed.

JSHint passed.

reviewboard/static/rb/js/accountPrefsPage/views/apiTokensView.ts (Diff revision 1)

The issue has been resolved. Show all issues

Thoughts on using this format instead?
super.initialize(_.defaults(options, {
    size: DialogSize.LARGE,
}));

I feel like I've seen us pass parent options like that elsewhere in the codebase.

Commits:

	Summary	ID
	Update the API token policy editor from $.modalBox to Ink.DialogView. This change ports the API token policy editor pop-up from our old `modalBox` function to the new `DialogView`. Testing Done: - Ran js-tests. - Verified that the functionality and appearance of the editor dialog worked as expected.	5cf92490785a31da4b6c6763246ad42540cedceb
	Update the API token policy editor from $.modalBox to Ink.DialogView. This change ports the API token policy editor pop-up from our old `modalBox` function to the new `DialogView`. Testing Done: - Ran js-tests. - Verified that the functionality and appearance of the editor dialog worked as expected.	942b462c41ffa2f8a2cc3d7b2439c486556ccd0c

Diff:

Revision 2 (+338 -170)

Show changes

	reviewboard/static/rb/css/pages/my-account.less
	reviewboard/static/rb/js/accountPrefsPage/views/apiTokensView.ts

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```