8000 Fix remove ps by zhangshanwen · Pull Request #963 · qor5/admin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix remove ps #963

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.

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

Merged
merged 172 commits into from
May 22, 2025
Merged

Fix remove ps #963

merged 172 commits into from
May 22, 2025

Conversation

zhangshanwen
Copy link
Contributor

No description provided.

molon and others added 30 commits November 12, 2024 17:39
This commit fixes the style issues introduced in e49b1d8 according to the output
from Gofumpt.

Details: #748
This commit fixes the style issues introduced in 4e2998f according to the output
from Gofumpt.

Details: #779
This commit fixes the style issues introduced in 84b5f36 according to the output
from Gofumpt.

Details: #786
- Modified multiple test cases across various files to replace `nil` with `http.NoBody` in `httptest.NewRequest` calls for improved clarity and consistency in handling empty request bodies.
- This change enhances the readability of the test code and aligns with best practices for HTTP request handling.
- Updated the test cases in `fields_test.go` to replace string length checks with direct string comparisons for improved readability.
- Modified the HTTP request creation in tests to use `http.NoBody` instead of `nil`, aligning with best practices for handling empty request bodies.
- Updated the test in `fields_test.go` to replace string length checks with direct string comparisons for enhanced clarity in error reporting.
- This change aligns with previous refactoring efforts to simplify test assertions and improve code readability.
- Eliminated redundant return statements in multiple test cases within `demo_case_test.go`.
- This change enhances code clarity and maintains consistency with previous refactoring efforts to simplify test assertions.
zhangshanwen and others added 23 commits May 13, 2025 10:17
…n support

- Added a new function to dynamically set the editing title in the page builder based on the model ID and locale.
- Updated the rename container dialog to include a script for setting the right content title.
- Changed confirmation message keys in the publish messages to improve clarity and consistency across different languages.
- Updated external asset links in the page layout for better versioning control.
- Added localization support for "no records to show" message in the related online pages tab of the page builder.
- Introduced a new boolean flag to determine if online pages are present, improving the conditional rendering logic for the tab's content.
- This change enhances user experience by providing clearer feedback when no related online pages are available.
refactor: enhance related online pages tab configuration
This reverts commit b1a5a6a.
- Modified the container wrapper in the editor to include an additional class "inner-container" for improved styling.
- Added CSS to disable pointer events for the "inner-container" class, enhancing user interaction control in the editor.
- Introduced a new boolean flag `disabledShared` in the Builder struct to control the visibility of the "Mark as Shared" option in the UI.
- Updated the rendering logic in `model_events.go` to conditionally display the sharing option based on the new flag, enhancing user experience by providing clearer control over container sharing.
This commit fixes the style issues introduced in d994030 according to the output
from Gofumpt.

Details: #963
- Removed the unnecessary class "inner-container" from the container wrapper when in readonly mode.
- Changed the label "Add Row" to "Add Item" in English and Japanese message files for better clarity.
This commit fixes the style issues introduced in b5a92dd according to the output
from Gofumpt.

Details: #963
@zhangshanwen zhangshanwen merged commit 9a72d1c into main May 22, 2025
9 checks passed
@zhangshanwen zhangshanwen deleted the fix-remove-ps branch May 22, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0