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

[Stage 1] clarify requirements of the nonograms task #1751

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

asfound
Copy link

@asfound asfound commented Feb 7, 2025

Title of Pull Request

🤔 This is a ...

  • 🌟 New task
  • 🌐 New module
  • ⚙️ Update to an existing task
  • 🔧 Update to an existing module
  • 🔗 Update or addition of external resources or links
  • 🐛 Fix in a task or related content
  • 🛠 Fix in a module or related content
  • ✏️ Fixed a typo or grammatical error
  • 🔗 Fixed a broken link
  • ❓ Other (specify: ****____****)

Description

  • Brief Overview:
    Clarified the requirements for the task based on discussions of students in the Discord channel. Required functionality and game states were specified. Technical requirements were also clarified.
  • Implementation Approach:
    Refined the specific behaviour expected from various game features. This includes ensuring accurate stopwatch functionality, save/reset/continue buttons functionality, time format, sound requirements and providing clearer instructions for the user interface. Additionally, some typos were corrected to improve clarity, and obvious game rules were explicitly defined to ensure better understanding.

Additional Information

  • Screenshots/Links:
  • Related Issues:

Checklist

  • ✅ I have performed a self-review of my own code.
  • 📝 I have commented my code, particularly in hard-to-understand areas.
  • 🔧 I have made corresponding changes to the documentation (if applicable).
  • 🚫 My changes generate no new warnings or errors.

@asfound asfound changed the title fix: clarify functional requirements of the nonograms task fix: clarify requirements of the nonograms task Feb 7, 2025
@asfound asfound changed the title fix: clarify requirements of the nonograms task docs: clarify requirements of the nonograms task Feb 7, 2025
@stardustmeg
Copy link
Collaborator

Please, check the formatting and run Prettier

@stardustmeg stardustmeg changed the title docs: clarify requirements of the nonograms task [Stage 1] clarify requirements of the nonograms task Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants