sillsdev / sillsdev/languageforge-lexbox
Picture model should include PublishIn
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 9
- Forks
- 8
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 49
Description
Describe the feature
The MiniLcm model for Pictures in https://github.com/sillsdev/languageforge-lexbox/pull/2300 didn't include the PublishIn field. But it's exposed in the FW UI, and documented in the online help: you can choose to publish pictures in potentially a different set of publications than their parent sense (e.g., you might want to publish them in the online dictionary but not in the printed one). This is functionality that some users will definitely want, so we'll need to update the MiniLcm Picture model to include a PublishIn field.
Who is this feature for?
Users
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the MiniLcm Picture model and compare it with the fields exposed in the FW UI and described in the online help. Add the PublishIn field so pictures can be assigned to a different set of publications than their parent sense, then verify that the model exposes this functionality.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 70/100