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

/stuck: Command that shows diagram for when people query about the best "I'm stuck" workflow #466

Closed
3 of 8 tasks
MaoShizhong opened this issue Oct 23, 2023 · 7 comments · Fixed by #497
Closed
3 of 8 tasks
Assignees

Comments

@MaoShizhong
Copy link
Contributor

MaoShizhong commented Oct 23, 2023

Complete the following REQUIRED checkboxes:

  • I have thoroughly read and understand The Odin Project Contributing Guide

  • The title of this issue follows the command name: brief description of request format, e.g. /help: add optional @user parameter

The following checkbox is OPTIONAL:

  • I would like to be assigned this issue to work on it

1. Description of the Feature Request:
It's quite common for people to ask questions about or in some way express frustration that highlights or raises suspicion about less-than-ideal learning habits and expectations during exercises/projects. Quite commonly they'll be along the lines of not "fully understanding" everything after reading a lesson, or feeling like they did but as soon as they got to a project it's out the other ear already, not revisiting past lessons, or something that hints at them reluctant to ask for help/thinking asking for help should only ever be a last resort and they should struggle alone as long as is possible first.

@bycdiaz drew up a very neat little diagram the other day which I think (perhaps along with a brief text-version), would be really useful to put in an embed callable by /stuck or some equivalent/better name. It should help streamline and make giving advice for this a little more consistent.
Maybe also add the link to AI and learning at the very bottom of it just mentioning the dangers there.

2. Acceptance Criteria:
A list of checkbox items that explain the requirements needed to be met to resolve this request, e.g.:

@MaoShizhong MaoShizhong added the Status: Needs Review This issue/PR needs an initial or additional review label Oct 23, 2023
@ManonLef
Copy link
Member

Sounds wonderful Mao! I'd love to hear the copyright holder's thoughts @bycdiaz on this as well 😄.

I am of the opinion it could be useful to add a note on the /question template command because many people will end up asking a question when stuck. Could you expand on your motivation for including note of the /ai command?

@MaoShizhong
Copy link
Contributor Author

The link from/mention of the content from /ai I feel would be useful here to emphasise and clarify that using AI as a legitimate step in the problem solving process (in this situation where people are doing this to learn and not complete work tasks on a deadline) is not advisable for the reasons that link specifies.
I've often found value in using the /ai command after talking about more effective ways to approach problem solving, either as a side note or in response to the other person asking about/suggesting its use. Therefore if that little bit is included at the bottom of a /stuck command, I feel it wouldn't be out of place at all.

@bycdiaz
Copy link
Member

bycdiaz commented Nov 4, 2023

I wouldn't object to including the diagram at all :)

@MaoShizhong
Copy link
Contributor Author

If this does go ahead, would the preferred way to handle the image be by uploading it elsewhere then linking the URL in the command embed? Or by using the attachment builder which will upload the image using a local file directly?

@MaoShizhong
Copy link
Contributor Author

Gentle bump
Had a lot of cases over the last few months where I've linked the workflow message/image which has shown to be really helpful.

@MaoShizhong
Copy link
Contributor Author

@bycdiaz @CouchofTomato Just wanted to clarify if the assignment to Carlos was intended/means the idea is approved and he's been asked to work on implementing this.

Based on his reaction on Discord, there may or may not have been a miscommunication?

@01zulfi
Copy link
Member

01zulfi commented Feb 24, 2024

@MaoShizhong, like the idea, you're assigned.

@01zulfi 01zulfi removed the Status: Needs Review This issue/PR needs an initial or additional review label Feb 24, 2024
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 a pull request may close this issue.

4 participants