Pull Request for our meeting tomorrow (3/8/24)

[anschelburk]

Steps taken:

• Removed global variables from within functions. (They are still being used as function inputs, but we can address this on our call tomorrow morning.)
• Removed format_daily_values_dict - I created it previously but found a way to avoid using it. Deleted it since it’s unused.
• I think you may have mentioned you weren’t sure, in the format_json_data_as_dict() function, why I created and returned a variable called json_data_formatted. Renamed the input variable and added a docstring to make this clearer.
• Removed “subtract daily_value_dicts” function, since it’s not being used anymore. (It’s been replaced with a more updated merge_dicts function.)
• Combined get_search_results_data(), update_search_results_name(), update_search_results_nutrients() functions into a single, larger function: update_search_results_name_and_nutrients(). This should simplify the code, and pave the way for combining the results_name and results_nutrients variables into a larger dict.
• Fixed dropdown list issue - no more green numbers, and formatting is now human-readable (if a little different than I originally had planned).

Questions:

• I want to add general_scratch_code.py to .gitignore. Can I just do this, or does it need to be on a new branch? Let’s talk about this issue on our call tomorrow - I think it may be helpful to talk in person.
• Let’s talk about the global variables - I feel like I need them, but am not sure how exactly to define them.

Next Steps:

• Big Step: refactor code to combine results_name and results_nutrients into a single larger dict. Then:
• Modify ‘remove ingredient from list’ button so that it’s under ‘My Ingredients List’
• Create multiplication function
• Combine add, subtract, and multiplication functions into one larger function using enum

[bbelderbos]

Great progress update, I am reviewing the code, talk in a bit ...