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

Group 10 Final Submission #7

Open
wants to merge 87 commits into
base: main
Choose a base branch
from
Open

Group 10 Final Submission #7

wants to merge 87 commits into from

Conversation

mgovilla
Copy link

No description provided.

@charlieroberts
Copy link
Contributor

Great idea! I'm not sure what you're interested in using for your rich text editor widget, but I really like ProseMirror.

One aspect that used to frustrate me with resumes in word processing docs was an inability to disable certain sections... for example, I don't want to post a resume to my website with my personal phone number or address, but obviously need to include this for applying to jobs. So being able to remix resumes for different applications makes a lot of sense to me.

I'm not sure how you're dividing up the work, but if at all possible I would try to tackle the various stages one at a time, and make getting through the "Remixing" page your baseline goal. I like the idea of a review session (a lot!) but would classify this as a stretch goal given the short timespan we have for the project. That said, if you can get to it, go for it! And if you really want to get fancy, you could go for a collaborative editor, so that reviewers could make simple edits, using yjs + prosemirror: https://docs.yjs.dev/ecosystem/editor-bindings/prosemirror

Good luck on this, and I'm looking forward to seeing how far you can take it!

mgovilla and others added 27 commits October 11, 2021 03:15
updated style of remix page and added buttons
@mgovilla mgovilla changed the title Group 10 Proposal Group 10 Final Submission Oct 11, 2021
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.

5 participants