Skip to content

Wings Update Alert (Reworked)#5655

Open
itzlinaton wants to merge 7 commits into
pterodactyl:1.0-developfrom
itzlinaton:1.0-develop
Open

Wings Update Alert (Reworked)#5655
itzlinaton wants to merge 7 commits into
pterodactyl:1.0-developfrom
itzlinaton:1.0-develop

Conversation

@itzlinaton

@itzlinaton itzlinaton commented Jun 16, 2026

Copy link
Copy Markdown

Changes:

  • Introduced a simple alert similar to the panel update alert for Wings by attempting to follow the Pterodactyl Code Standard.
{89CD811D-41F2-4238-B3FB-DA6266F5392E}
{35201C06-EC73-4116-BB42-0373DBB26ABB}

@lajczi lajczi 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.

cool, now looks good. Thank you and now please wait patiently for merge

@itzlinaton

Copy link
Copy Markdown
Author

cool, now looks good. Thank you and now please wait patiently for merge

Awesome!

@AndyIsHereBoi

Copy link
Copy Markdown
Contributor

I like this look a lot more, fits nicely with the panel warning

@itzlinaton

itzlinaton commented Jun 16, 2026

Copy link
Copy Markdown
Author

I like this look a lot more, fits nicely with the panel warning

Thanks, I tried to take your advice.

@AndyIsHereBoi

Copy link
Copy Markdown
Contributor

Looking at this again, the default panel already shows current and latest ones, why was all the other stuff added in?

mainly what im saying is couldnt we just exclude the v at the fronts and then compare the 2 strings to be the same?

@itzlinaton

itzlinaton commented Jun 17, 2026

Copy link
Copy Markdown
Author

mainly what im saying is couldnt we just exclude the v at the fronts and then compare the 2 strings to be the same?

Also possible however I tried to keep it like how it was before with minimal changes.

- Made permission button use "TitledGreyBox" container instead
- Added a new component (AllPermissionsButton.tsx) to make code cleaner
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.

3 participants