Skip to content

[EXTERNAL] fixed a wrong data type given in the instructions.#3016

Merged
pedrodesu merged 3 commits into01-edu:masterfrom
madaghaxx:master
Sep 2, 2025
Merged

[EXTERNAL] fixed a wrong data type given in the instructions.#3016
pedrodesu merged 3 commits into01-edu:masterfrom
madaghaxx:master

Conversation

@madaghaxx
Copy link
Copy Markdown
Contributor

The instruction were requesting a HashMap with a tuple of strings as key and a value of a callback function but the expected function used a HashMap with string as key and callback as value.
The given instructions where misleading the users.

The instructions:

  • A second structure named FlagsHandler will be given which just has one element: flags: HashMap<(String, String), Callback>. You'll also need to implement the following associated functions:

The given expected function:

pub struct FlagsHandler {
    pub flags: HashMap<String, Callback>,
}

So the fixed version of the instructions is :

  • A second structure named FlagsHandler will be given which just has one element: flags: HashMap<String, Callback>. You'll also need to implement the following associated functions:

@madaghaxx madaghaxx closed this Aug 2, 2025
@madaghaxx madaghaxx reopened this Aug 2, 2025
@madaghaxx
Copy link
Copy Markdown
Contributor Author

Fixed the raid subject with 2 main keys:

  • Added the instruction clarifying that shapes must be in different colors because nothing in the instructions indicates it and it is mandatory in the audit page.
  • Fixed the main function to give an image close to the one in the instructions.

@HarryVasanth HarryVasanth changed the title fixed a wrong data type given in the instructions. [EXTERNAL] fixed a wrong data type given in the instructions. Aug 27, 2025
@pedrodesu
Copy link
Copy Markdown
Contributor

pedrodesu commented Sep 2, 2025

Hello @madaghaxx! Thank you for the PR. Everything looks good, and so we will merge this. The action failed but it's a false positive. We will report this too.

Let me know if you have anything else to add. Thank you!

@pedrodesu pedrodesu merged commit 53cb846 into 01-edu:master Sep 2, 2025
7 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗂️ docs Documentation 📕 Rust Rust

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants