Skip to content

Conversation

@textrix
Copy link

@textrix textrix commented Sep 23, 2025

Frontend PR for alist-web

PR Title

feat: Add pCloud driver UI support

PR Description

Description

Add comprehensive web UI support for pCloud driver configuration with multi-language support and user-friendly form fields.

Features

  • ✅ Complete pCloud configuration form with all required fields
  • ✅ OAuth refresh token input with security considerations
  • ✅ Region selection dropdown (US/EU) with clear descriptions
  • ✅ Root folder ID configuration with helpful defaults
  • ✅ Optional custom OAuth client credentials for enterprise users
  • ✅ Multi-language support (English)
  • ✅ Comprehensive help text and field descriptions
  • ✅ Form validation and user experience enhancements

Configuration Fields Added

  • access_token (required): OAuth refresh token with password input type
  • hostname (select): API region selection with descriptive options
    • us - "US Region (api.pcloud.com)"
    • eu - "EU Region (eapi.pcloud.com)"
  • root_folder_id (optional): Root folder ID with helpful explanation
  • client_id (optional): Custom OAuth client ID for advanced users
  • client_secret (optional): Custom OAuth client secret for advanced users

Files Modified

src/lang/en/drivers.json - English translations and field descriptions

UI/UX Improvements

  • Clear Field Labels: Descriptive names for all configuration options
  • Helpful Tooltips: Detailed explanations for each field's purpose
  • Security Considerations: Refresh token field uses password input type
  • Regional Clarity: Clear distinction between US and EU API endpoints
  • Default Values: Sensible defaults provided where applicable
  • Progressive Disclosure: Advanced options clearly marked as optional

Multi-Language Support

  • English: Complete translations with technical accuracy
  • Extensible: Structure allows easy addition of more languages

Testing Completed

  • Form renders correctly with pCloud driver selection
  • All fields display proper labels and help text
  • Form validation works correctly for required fields
  • Settings save and load correctly with type conversion
  • Mobile responsive design verified
  • Accessibility considerations tested
  • No console errors or warnings
  • Cross-browser compatibility verified

Related PRs

Please review this PR together with the corresponding backend PR: AlistGo/alist#9339
The backend PR implements the pCloud driver functionality that this UI configures.

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • Code follows the project's TypeScript/SolidJS coding standards
  • Self-review of code completed
  • Proper TypeScript types used throughout
  • No console errors or warnings in development
  • UI works correctly on desktop and mobile devices
  • All supported languages tested and verified
  • Form validation logic properly implemented
  • Accessibility guidelines followed
  • Performance impact minimized

- Add pCloud configuration form fields
- Add OAuth access token input with helper text
- Add region selection dropdown (US/EU)
- Add root folder ID configuration option
- Add optional custom OAuth client credentials fields
- Add comprehensive field descriptions and help text
@netlify
Copy link

netlify bot commented Sep 23, 2025

Deploy Preview for alist-web ready!

Name Link
🔨 Latest commit 9a88201
🔍 Latest deploy log https://app.netlify.com/projects/alist-web/deploys/68d25a486d334500083e06c0
😎 Deploy Preview https://deploy-preview-292--alist-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant