Skip to content

KNOX-3224: Upgrade Token Management and Token Generation UI #1118

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.

Sign up for GitHub

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

Jump to bottom
Merged
hanicz merged 9 commits into from
Dec 18, 2025
Merged

KNOX-3224: Upgrade Token Management and Token Generation UI #1118

hanicz merged 9 commits into from
Dec 18, 2025

Conversation

@hanicz
Copy link
Contributor

@hanicz hanicz commented Dec 4, 2025 •
edited
Loading

KNOX-3224 - Upgrade Token Management and Token Generation UI

What changes were proposed in this pull request?

Token Generation, Token Management

  • Upgrades the Token Generation UI and Token Management UI to bootstrap 5+, angular 21+ and node to v22.20, sweetalert to 11.26.3, jquery to 3.7.1
  • Restructured the project
  • Replaced old toPromise calls with firstValueFrom
  • Removed old slow prebuilt deep-purple style and added new green style

Homepage

  • Upgrades the Homepage UI to angular v21
  • Changed default show proxy information to true
  • Fixed invalid profiles call (duplicate '/' in URL)
  • Changed deprecated HttpClientModule to provideHttpClient

How was this patch tested?

rm -rf package-lock.json node_modules/
npm install
Screenshot 2025-12-15 at 18 22 59 Screenshot 2025-12-15 at 18 23 09 Screenshot 2025-12-08 at 19 07 17

hanicz added 3 commits December 8, 2025 07:49
@hanicz
KNOX-3224: Refactor token generation UI, upgrade node ,angular, boots…
58e202b 
…trap, jquery and other dependencies
@hanicz
KNOX-3224: Upgrade Token Management UI
7c37c4b
@hanicz
KNOX-3224: Replaced old token-management prebuilt theme, Changed defa…
d2b5bb7 
…ult show proxy information to true, Fixed invalid profiles call (duplicate '/' in URL)
@hanicz hanicz self-assigned this Dec 8, 2025
@hanicz hanicz added the angular label Dec 8, 2025
@hanicz hanicz marked this pull request as ready for review December 8, 2025 06:52
@github-actions
Copy link

github-actions bot commented Dec 15, 2025 •
edited
Loading

Test Results

7 tests   7 ✅  1s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 94eeed4.

♻️ This comment has been updated with latest results.

smolnar82
smolnar82 approved these changes Dec 18, 2025
View reviewed changes
Copy link
Contributor

@smolnar82 smolnar82 left a comment

Choose a reason for hiding this comment

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

LGTM.
Thanks, @hanicz, for taking care of this!

@hanicz hanicz merged commit eb4c2c7 into apache:master Dec 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@smolnar82 smolnar82 smolnar82 approved these changes

@moresandeep moresandeep Awaiting requested review from moresandeep

@pzampino pzampino Awaiting requested review from pzampino

@bonampak bonampak Awaiting requested review from bonampak

Assignees

@hanicz hanicz

Labels

angular

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hanicz @smolnar82