Swati Dewan proposed openstack/solum-dashboard: Fix the applications create functionality in the dashboard https://review.openstack.org/349279
vijendar has quit
devkulkarni1 joined the channel
devkulkarni1 has quit
devkulkarni1 joined the channel
zhurong joined the channel
zhurong has quit
zhurong joined the channel
devkulkarni1 has quit
julim joined the channel
zhurong_ joined the channel
zhurong has quit
itisha has quit
pt_15 has quit
julim has quit
openstackgerrit joined the channel
itisha joined the channel
pt_15 joined the channel
pt_15
Hi zhurong_ :)
zhurong_
pt_15 hi
pt_15
zhurong_: I made the changes in the app create patch, and submitted it. However, I am still having a problem with the file fields. At least on my setup, the file field always shows None, even if I do attach a file. Could you please take a look at that and tell me if its creating a problem in your system too? And if it is, could you tell me what is the
mistake I am making in that?
zhurong_
pt_15 sure I will check tonight or tomorrow, Here we are 5:30pm
pt_15
zhurong_: sure, not a problem. Thanks :)
zhurong_ has quit
zhurong joined the channel
julim joined the channel
pt_15 has quit
itisha has quit
openstackgerrit joined the channel
mkam joined the channel
zz_dimtruck is now known as dimtruck
mkam has quit
zhurong has quit
mkam joined the channel
devkulkarni1 joined the channel
devkulkarni1
good morning / good evening
pt_15 joined the channel
pt_15
Hey devkulkarni1 :)
itisha joined the channel
devkulkarni1
hi pt_15
Ablu
Hi
pt_15
Hi Ablu!
devkulkarni1: I've submitted a new patchset for app create: https://review.openstack.org/#/c/349279/ which now takes the choice of whether to provide an app file or provide per-field input
devkulkarni1
pt_15: nice.. so you were able to resolve the issue that you were running into earlier?
pt_15
devkulkarni1: no, I still have a problem in reading the file, what happens when i try and give the app file is that it says you haven't provided it, you need to as it is required. Basically the FileInput is not able to actually take in any value
it could just be an issue with my setup, as I checked out similar code in the murano dashboard, and i couldn't really see where it could be going wrong
devkulkarni1
pt_15: in that case, I would suggest you disable/remove the app file input from the above patch. that way we won't be merging functionality that is not yet working. you can then submit a follow up patch to add support for app file input
pt_15
devkulkarni1: oh ok, cool
devkulkarni1: although, if possible could you test this part out, so that we can at least know if its a problem with my setup, or with the code itself?