Replies: 3 comments 5 replies
-
Do we want to change this in the MR or as a separate issue afterwards? I ask because this is only a very small improvement. If an issue is the right place, I'll open one. Otherwise I can describe the suggested changes in more detail here. |
Beta Was this translation helpful? Give feedback.
-
And one further question: Is there a place where we can record this information? Is there anything like "release notes" or similar? Some place that might be read when updating to a new version? |
Beta Was this translation helpful? Give feedback.
-
Referring to @RayOei 's questions here:
#1073 (comment)
I don't really think we need to describe how the URL may look like - it can be any site really. All it needs to do is respond compliant with the OpenAPI definition - see the description here.
But I do agree, that in this case some explanatory text or even a link to the documentation next to the field might be helpful. If there are any specific suggestions, we could create an issue for it.
It's not just internal - any application with the link may use the supplier shared through this link. (Again, see the description linked above.)
The idea is that you could have both: A central instance of foodsoft, that serves for nothing but sharing suppliers (just as sharedlists currently does) or you could have "normal" foodsoft instances, which share single suppliers amongst them.)
You could be right. To be honest, we could never reliably who uses it for what. 😅 From an architectural point of view I think it was meant to do what its name suggests: Provide a centralized list of suppliers with their articles, that can be synchronized to an infinite number of foodsoft instances. That's the part that, which we reimplemented - this time directly in foodsoft itself. How the foodcoops usually filled in the lists (whether they used the import formats it provided or just manually entered them), we do not know.
Yes, this is one of the things we thought, we could do after the merge (to not further bloat the PR) - see the old issue about it:
#1100
Beta Was this translation helpful? Give feedback.
All reactions