Skip to content
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

[UTOPIA-1583] Implement package for text editor on PIA Pathways Questionnaire #1795

Merged
merged 5 commits into from
Nov 18, 2023

Conversation

JieunSon96
Copy link
Contributor

🎯 Summary

UTOPIA-1583

  • Updated jsonb types in the backend and updated mocks.
  • Created migration.
  • Replaced text editor on PPQ tab.
  • Ensured tests pass in the backend.
  • Added state change handler function for the text editor.
  • Added character limit for "pidInitiativeSummary" text editor but should be fixed more for cursor issue(the cursor moves to the beginning when the character limit of 500 is reached)

🔰 Checklist

  • I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation where required.
  • I have tested my changes to the best of my ability.
  • I have consulted with the team if introducing a new dependency.
  • My changes generate no new warnings.

@JieunSon96 JieunSon96 self-assigned this Nov 17, 2023
@github-actions github-actions bot added DPIA API PR contains changes to DPIA API DPIA Web App PR contains changes to DPIA frontend app labels Nov 17, 2023
@JieunSon96 JieunSon96 merged commit acc5a21 into main Nov 18, 2023
14 of 15 checks passed
@JieunSon96 JieunSon96 deleted the UTOPIA-1583 branch November 18, 2023 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DPIA API PR contains changes to DPIA API DPIA Web App PR contains changes to DPIA frontend app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants