Skip to content

fix(web-app, admin): Angular 21+ modernizing for feeds & authentication - #507

Open
ktsorensenWX wants to merge 2 commits into
developfrom
fix/admin-delete-dialogs-angular-upgrading
Open

ktsorensenWX wants to merge 2 commits into
developfrom
fix/admin-delete-dialogs-angular-upgrading

Conversation

@ktsorensenWX

@ktsorensenWX ktsorensenWX commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Continuing angular 21+ modernizing by enabling standalone for admin authentication & feeds
  • Signals were introduced where needed
  • admin.module.ts was updated to remove old imports.

Testing

  • Tests using npm returned 1043 succeses

To-Do

  • Adding comments
  • Web UI testing

@jclark118 jclark118 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you also update the button style icons/colors to match the other pages while you're in here? See the users page for an example. All the stuff in this PR looks good though
Screenshot 2026-09-18 at 11 13 12 AM
Screenshot 2026-09-18 at 11 12 58 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants