sillsdev / languageforge-lexbox

Lexbox, SIL linguistic data hub
MIT License
7 stars 2 forks source link

Viewer release #797

Open myieye opened 4 months ago

myieye commented 4 months ago

Final TODOs before we can release the viewer:

hahn-kev commented 4 months ago

For feedback I 100% want this as a feature and I like the idea of a google form, however there's no way to pass metadata to a google form (IE what the URL was where they clicked feedback), so I don't think that will work for us unless we want zero context on their feedback. That said I feel like a prefilled email is probably the simplest way to go, we can prefill the body of the email with a mailto link, then the user can just provide their feedback, we also get their email address even if they aren't a registered user. Just a thought, I'm sure there's services out there that will do it for us but I don't really want yet another tool, and anything else we already have would require an account. The only other option would be that we build our own feedback feature (since they already have an account with us) but I don't want to spend the time on that.

megahirt commented 4 months ago

I like the idea of a google form, however there's no way to pass metadata to a google form (IE what the URL was where they clicked feedback)

Actually, ChatGPT says you can pre-populate a google form with URL encoded GET values and assigning them to form elements using their IDs (which you get by inspecting the form). https://chat.team-gpt.com/lt-lexical-tools/6656ebe4ca989092547d2c1e

hahn-kev commented 4 months ago

Based on a stack overflow post it's not possible, so I suspect the AI might be making that up, can you test it to verify if it's correct?

myieye commented 4 months ago

You can edit the query param values in this URL and it works: https://docs.google.com/forms/d/e/1FAIpQLSe8p-hiEZZHse3yEvqVQnAYVoFHdBRVPTuLXISC4Q8F8u_ULw/viewform?usp=pp_url&entry.1828162391=Test+tim+edited&entry.1256080784=Option+3

hahn-kev commented 4 months ago

oh awesome. I just realized I made an issue #637 for this already, so I'll copy your response into it