casperlundberg / dynamic-monitoring-system

1 stars 0 forks source link

set_param_in must be better #11

Open casperlundberg opened 1 month ago

casperlundberg commented 1 month ago

currently it just assumes that all parameters are of one type but they can be multiple types. They can also be non-url types such as headers and cookies but make it work for both pathparam and queryparam first, this means that the parameter system in RequestClass needs to be updated as well, maybe something like:

parameters = {[name, value, "path"], [name2, value2, "query"]}

order doesn't matter, just the point that different formatters need to be used for different url params

casperlundberg commented 3 weeks ago

This is reworked to be a check that the ui does when listing the parameters enums or types to ensure that params are correctly filled in.

casperlundberg commented 3 weeks ago

Fix typecheck at some point for the non enum params, not needed the attention of being pinned anymore

casperlundberg commented 1 week ago

When filling in incorrect types the ui softlocks due to the response being of an type we cant handle (not 200)