Skip to content

feat: RSS-SE-16 synchronize editor and viewer - #15

Merged
FierceSloth merged 26 commits into
feature/swaggerEditorfrom
story/RSS-SE-16_synchronizeEditorAndViewer
Jul 3, 2026
Merged

FierceSloth merged 26 commits into
feature/swaggerEditorfrom
story/RSS-SE-16_synchronizeEditorAndViewer

Conversation

@FierceSloth

Copy link
Copy Markdown
Owner

📋 Trello Task ID

  • RSS-SE-16 (Task ID / Trello Card)
  • No ID (Global fix or chore)

⚡️ Summary

  • Refactor SwaggerEditor to operate as a fully controlled component by removing its internal local state.
  • Implement a custom useDebounce hook to optimize performance during user input.
  • Lift state management up to the HomePage to track raw text and validation status.

🛠 Type of change

  • feat (New feature)
  • fix (Bug fix)
  • refactor (Code improvement / Refactoring without changing logic)
  • style (Formatting, CSS)
  • docs (Documentation)
  • chore (Configs, Build)
  • test (Testing)

@vercel

vercel Bot commented Jul 3, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
swagger-editor-app Ready Ready Preview, Comment Jul 3, 2026 5:50pm

const [localValue, setLocalValue] = useState(externalValue || '');

const [validation, setValidation] = useState({ errors: 0, warnings: 0 });
const isValid = validation.errors === 0 && currentFormat !== 'unknown';

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

не нужно ли проверить на пустую строку? value.length > 0

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Нужно, спасибо за замечание

@FierceSloth
FierceSloth changed the base branch from story/RSS-SE-15_createMainPageResponsiveSplitView to feature/swaggerEditor July 3, 2026 17:38
@FierceSloth
FierceSloth merged commit 5d80c2f into feature/swaggerEditor Jul 3, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
Preview — 63ca3798 Deployed Jul 3, 2026 by vercel[bot]
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