Rename toVueObject -> toReactObject on User model - #5610
Open
camwhi-te wants to merge 36 commits into
Open
Conversation
Contributor
|
@camwhi-te Can you refactor this to |
This fixes an overflow bug that occurs on smaller screens on the ApiKeyModal opening. **Context**: Whenever you create an `Client Api Key` the `ApiModal` is opened on the user's interface and the key is displayed, but as it wasn't correctly *overflow-protected* it overflows on smaller screens. - Before:  - After:  Closes pterodactyl#5625
The admin node view renders the daemon-supplied cpu_count without escaping. Coerce it to an integer so untrusted node telemetry cannot inject markup. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The configuration endpoint returns the node's decrypted daemon token, so gate it behind write access instead of read. The shared single-node view request keeps read access. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Fixes a couple of UI/UX issues on the Admin Application API page
(`new.blade.php`):
- Mobile Layout: Fixed the radio buttons stacking vertically and
breaking the page on small screens. The table now scrolls horizontally
and stays clean.
- Button States: The bulk select buttons ("Read All", etc.) now
dynamically change color to indicate which option is currently active.
Screenshots:
- Before:
- After:
Co-authored-by: Robert Dennis <31261583+robertdrakedennis@users.noreply.github.com>
Replace references to the Laravel 5.7 validation docs with links to the 11.x documentation in resources/views/admin/eggs/variables.blade.php <img width="374" height="25" alt="image" src="https://github.com/user-attachments/assets/38090fe3-6ed8-4a73-b700-f8ce8d3060ab" />
Currently, when a node is under maintenance, the server row keeps loading indefinitely. This means that there will be no requests to the resource API, and the user will be able to know this without directly accessing the server. old: <img width="1347" height="603" alt="tempsnip" src="https://github.com/user-attachments/assets/523b6321-6ecc-45d2-8cb4-b6e14a473a44" /> new: <img width="1258" height="618" alt="tempsnip2" src="https://github.com/user-attachments/assets/5f4b77d5-1c49-4f30-a068-a96f8bc83177" /> (I reopened it because I decided to delete my fork to clean things up. And use better techniques)
fixes pterodactyl#5576 --------- Co-authored-by: MrSoulPenguin <28676680+MrSoulPenguin@users.noreply.github.com>
This simply adds a link to [this page](https://pterodactyl.io/panel/1.0/updating.html) when the panel isn't up to date.
…#5644) ### Description User creation was not properly logged within the action log, neither from sub-user invite or manual user creation. This PR adds the relevant log and includes testing. ### Manual testing scenarios 1. Invite a user within any created server 2. Create a user in the admin 3. Verify both users appear as an action in the action log ### Questions or comments AI was used for secondary testing, QA and code review. No code was written by AI. ### Resolved issues: 1. [x] resolves pterodactyl#5631
…ilename (pterodactyl#5637) Fix for pterodactyl#5630 The compress_one string in resources/lang/en/activity.php referenced :file as the placeholder, but the backend passes the filename under a files array. This caused the activity log to render {file}} instead of the actual filename Changed :file to :files.0 to correctly resolve the first filename from the array. Solution was identified by the issue reporter (@marioenache).
…DB connection type. (pterodactyl#5688) ### Summary Quick one line fix that fixes the migration failure reported in pterodactyl#5683 Most installs load the database/schema/mysql-schema.sql and never have to replay these migrations, since the environment variable db_connection=mariadb was set it doesn't use the squashed schema and has to replay all migrations from 2016. This connection variable is supported though it seems as per https://github.com/pterodactyl/panel/blob/1.0-develop/config/database.php#L63 but it also defaulted above to mysql This field was updated to include the backticks as part of pterodactyl@8ca0989 but I'm unable to confirm why, if I've missed something I apologize but other references in this file seem to also use the barename only this one was quoted. ### Testing Built from the current v1.14.1 release branch Dockerfile. With the schema dump removed to force a full replay for testing with the mysql database connection aswell. - MariaDB 11.8.8 - All migrations passed - MySQL - All Migrations Passed Installs with the dump present are also unaffected by this modification. [Apologies for bad formatting, i'm terrible with markdown]
…tall scripts" setting is enabled (pterodactyl#5696) ### Description You're able to enable a "skip egg install scripts" per server, while still being able to reinstall a server. With this setting enabled, reinstalling a server has no functional use, the server is locked and shutdown, then simply unlocked immediately afterwards after a success signal from wings. This PR disables the reinstall button when skip install scripts is enabled for both the user and admin, with a little component telling you why you cannot reinstall. The reinstall button is not disabled if the server is in a status that reinstall could resolve (eg. failed install). ### Resolved issues: 1. [x] resolves pterodactyl#5306
pterodactyl/pterodactyl-cdn publishes https://cdn.pterodactyl.io/releases/latest.json, which every install polls to decide whether to show the update banner. It cannot read the new version from the API itself, because the release created above is still a draft and /releases/latest excludes drafts, so the tag is passed in the dispatch payload. Prerelease tags are skipped: version_compare on the panel side would tell every stable install to "upgrade" to a release candidate. GITHUB_TOKEN cannot be used here as it has no access to another repository, hence CDN_DISPATCH_TOKEN.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
13 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Simple change renaming Eloquent helper in accordance with actual UI stack.
Also, who decided to use Vue initially? ew.