Skip to content

Add initial content and data clearing for study tasks - #224

Merged
kcarnold merged 22 commits into
mainfrom
feat/pre-loading
Jun 24, 2025
Merged

Add initial content and data clearing for study tasks#224
kcarnold merged 22 commits into
mainfrom
feat/pre-loading

Conversation

@jooha-yoo

Copy link
Copy Markdown
Contributor
  • Implemented initial contents for task 2 and 3
  • Clears previous data when moving onto the next task page
  • Added taskID that differentiates between tasks, separate storage keys to store in localStorage
  • taskID in logging
    (Contains changes from main for post study task surveys)

Comment thread frontend/src/editor/editor.tsx
Comment thread frontend/src/editor/index.tsx Outdated
Comment thread frontend/src/editor/index.tsx Outdated
taskDescription: 'Task 2: Write a cover letter for the position described. The applicant is a recent college graduate with a major in Environmental Sustainability and a minor in Marketing, with relevant internship experience. Demonstrate how their background aligns with the company’s mission and requirements. [Details are given below in the editor document]',
initialContent: `GreenTech Solutions - Sustainability Coordinator Position

Company Overview:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

String has a bunch of embedded tab characters. This may or may not be a problem in practice.

Maybe declare this string outside of the render function.

Comment thread frontend/src/editor/index.tsx Outdated
@kcarnold

Copy link
Copy Markdown
Contributor

When we prepend the other info to the doc context, let's spot-check to make sure it has spacing / new lines / some sort of separator between the task description content and the person's document. Look at the prompt that we're actually giving the LLM and make sure it makes sense.

@jooha-yoo

Copy link
Copy Markdown
Contributor Author

@kcarnold could you take a look at the changes made in editor.tsx and index.tsx for pre-pending to docContext, professor? it seems to work fine- except it only seems to really be useful for 'Completion' suggestion type.

@kcarnold
kcarnold merged commit e9fb76b into main Jun 24, 2025
7 checks passed
@kcarnold
kcarnold deleted the feat/pre-loading branch June 24, 2025 20:46
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