-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pre-fill recording update on issues #444
Comments
@aarppe Do we want to add fields for lexical category and stem or just make available the |
All currently ticked changes have been deployed. |
Last two items should be a separate issue. |
A refactoring performed in 2021 changed names of fields, so the suggestion fields for this cases (likely for all issues before Sep 2021) are empty. There should have been a data migration added, but it looks like this was not the case. I will have to make a migration pass that tries to extract the information from the comment, which seems to be the only place where that information remains in the database. |
I was this close to saying "new issues should not have the problem", but it looks like they do too. I will look into this in further detail. |
Ok, in principle the information should be in the appropriate field, for the As a reminder for ourselves, we do also have a general |
Currently, new "wrong word" issues work, but not new "wrong speaker" issues, because wrong speaker issues only store the information in the comment. As we have to fix this anyways for old "wrong word", I'll do both on the same pass. |
(Transferred from incorrect issue) A fix is ready to deploy, but I'll wait until Daniel tells me he's done (or tomorrow morning, whichever happens first) to avoid issues. Just as a side note, Speaker Code suggestion is only available from June 2021 onwards. Word suggestions exist from almost the beginning. |
Updated entries on speech-db. |
@aarppe would you like the interface for issues to only show the field that should be changed in the issue? (say, only speaker or only phrase?), or keep it as it is? |
Storing this for reference. Sometimes the comment has the correct speaker code (here: ARL), but the pre-filled code is still the old one (here: BL2): https://speech-db.altlab.app/maskwacis/issues/5 ![]() ![]() |
Request: resolve all issues of the same kind when clicking on resolve. |
Change Merge phrases to entries. |
Merge for "câh-cakataham mîtosa namôya nikî-nipân" is not working with a 403 forbidden |
Must generate the "linguist-admin" role that can merge items. |
Recording update should have several improvements:
Exact same phrases (transcription+translations) should be merged automaticallyAn interface for manually merging phrases would be useful.See also: #443
The text was updated successfully, but these errors were encountered: