ZupIT/ritchie-formulas

Add required field in config.json

Closed this issue · 7 comments

lucasdittrichzup commented

What would you like to be added:

With the new required field added to the config.json file, it is interesting to adjust the formulas of the commons to adopt them in their respective files

Why is this needed:

Make the existing formulas compatible by adding the required field in the inputs.

Edit

Tasks:

  • Formulas group
  • Tasks
CptOrange16 commented

I could take this one

GuillaumeFalourd commented

Sure, I've assigned it to you @CptOrange16
Thank you for contributing ! 👍

CptOrange16 commented

@lucasdittrichzup @GuillaumeFalourd thank you for letting me help.
Can you help me understand exactly what needs to be done?
When you say:

With the new required field added to the config.json file, it is interesting to adjust the formulas of the commons to adopt them in their respective files
do you mean the tutorial field?

If so, how exactly would I introduce the tutorial field in the formulas?

GuillaumeFalourd commented

Sure @CptOrange16 👍

You'll have to update the config.json file of each formula inside the repository, and add the required field to each input set in the file.

You can find more informations on the documentation here

CptOrange16 commented

@GuillaumeFalourd hello, i've added the required field to the config.json files.
I wasn't sure what to write on the value so i wrote:

 "required": "Select true or false for this field."

If you want me to change that for any other value just let me know

GuillaumeFalourd commented

This field is a boolean value @CptOrange16 , so actually it's you who will have to inform if the inputs are required true or false.

ex: "required": true

CptOrange16 commented

This field is a boolean value @CptOrange16 , so actually it's you who will have to inform if the inputs are required true or false.

ex: "required": true

Aha yes thank you, that was what i was having doubts on. I'll get right on fixing that