Closed sseppi closed 4 months ago
@sseppi here's the updated version for file upload: https://www.figma.com/file/DmuP6Dbv5LzkCNOXrDnWIL/2023?type=design&node-id=1219%3A1707&mode=design&t=trlPGbjxC3ZpKG2J-1
@MatteoBiasi I think we are now ready for the estimation for the implementation also of this issue.
The latest PR is now on Test, i tested and what i noticed:
-On insert Adding a file and using "Save & next" calls the Save Method of the whole Data is this intentional? on edit it is working as expected
-On editing the Single Files ( i have 2 files inserted)
the "Update language settings" seems not to show the right data, it shows the same data on each file (Always the last inserted)
here for file 1 (where i inserted it + en)
and here for file 2 (where i only inserted en)
@sseppi please also test but as far i can see we have still some bugs here
@MatteoBiasi I got the same problems as Rudi reported, please test deeply the all fetures, fix the bugs and open a new PR once all bugs are solved.
It seems that now everything is working fine. As agreed this morning I ask @MatteoBiasi to deeply test the Save functionality considering also the problems mentioned during the call.
Before to close the issue: @pkritzinger a better and clearer text for the language activator popup @MatteoBiasi will change the texts and make a new PR
@sseppi @MatteoBiasi Here's a suggestion for the optimized text:
@pkritzinger the proposal is fine for me.
@MatteoBiasi I think you can implement the chnges.
@sseppi texts updated. Here's the PR: https://github.com/noi-techpark/it.bz.opendatahub.databrowser/pull/530
In the meeting of the 24th October we decided to keep the actual implemented version with the following changes:
NOTE: During the discussion it emerged that this component could be generalized to allow to easily make changes to fields that have a list of values (e.g. what does happen if in the future we add a lot of new languages?). The team decided to bring in production the actual version with the smallest changes as possible, to collect feedback from the users as soon as possible and with the little effort as possible. In any case, we should keep in mind that maybe it could be worth to consider a generalization of this component. (@mrabans please integrate this note or create a dedicated discussion if you think is necessary)
@pkritzinger can you please share the link to the updated version of the design?