Skip to content

Help Guide page creation#60

Draft
wskksw wants to merge 5 commits intomasterfrom
misc-change-staging
Draft

Help Guide page creation#60
wskksw wants to merge 5 commits intomasterfrom
misc-change-staging

Conversation

@wskksw
Copy link
Collaborator

@wskksw wskksw commented Dec 23, 2023

Description

just added a page for help guide that has videos and images.

Added a change to tailwind settings so that it doesn't overrule other styles.

Closes # (issue number)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code where needed
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

<iframe
width="560"
height="315"
src="https://www.youtube.com/embed/RjNERqpIQaw"

Choose a reason for hiding this comment

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

This video is outdated for new UI.
Could be confusing for new users.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is true though it seems the flow is mostly the same. Also, seeing as how we'll be making a lot more UI changes as the semester goes on, it might make sense to maybe leave updating the video for another time (and who knows, maybe if we need to make a video for our thesis or something, we could use that as the video)

Copy link
Collaborator

@AdamFipke AdamFipke left a comment

Choose a reason for hiding this comment

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

wrote some suggestions, you should just be able to click commit suggestion here on github. Also, I suppose it the page feels a little cramped in that some of the paragraphs and headers don't have very much margins. Also the navbar elements might need some horizontal margins. Rest looks good though, much better than the current 404 error or whatever it is right now haha

Comment on lines 94 to 95
<Panel header="async questions" key="4">
<p>Ask async questions in the course async queue</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<Panel header="async questions" key="4">
<p>Ask async questions in the course async queue</p>
<Panel header="Asynchronous Question Center" key="4">
<p>Have a question outside of office hours or lab hours? Hop in Asynchronous Question Center and ask you question there! The professor or TA will then be able to see and answer your question at their earliest convenience</p>

<p>Ask questions in the course queue, </p>
</Panel>
<Panel header="Chatbot" key="3">
<p>Chatbot will respond to your questions</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<p>Chatbot will respond to your questions</p>
<p>Chatbot will respond to your questions about the course or course content using professor-uploaded documents of the course</p>

></iframe>
<Collapse accordion>
<Panel header="Join a Course" key="1">
<p>Join a course using the course invite code</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<p>Join a course using the course invite code</p>
<p>Join a course using the course invite code provided by a professor</p>

<p>Join a course using the course invite code</p>
</Panel>
<Panel header="Ask Questions" key="2">
<p>Ask questions in the course queue, </p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<p>Ask questions in the course queue, </p>
<p>Ask questions in the course queue by entering a queue that has staff checked in and create a question. Once it's your turn, the TA/professor will come help you out</p>

</div>
</Content>

<Footer style={{ textAlign: 'center' }}>HelpMe ©2023 </Footer>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<Footer style={{ textAlign: 'center' }}>HelpMe ©2023 </Footer>
<Footer style={{ textAlign: 'center' }}>HelpMe ©2024 </Footer>

@Ferdinand737 Ferdinand737 self-requested a review February 6, 2024 17:45
@wskksw wskksw marked this pull request as draft February 9, 2024 19:56
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.

3 participants