vocascan / vocascan-frontend

A highly configurable vocabulary trainer
https://vocascan.com
Apache License 2.0
30 stars 8 forks source link

Mark duplicate Vocabs #128

Open noctera opened 2 years ago

noctera commented 2 years ago

Give a warning if vocab is duplicate:

  1. Option: Give a warning below foreign word text field image

  2. Option: When clicking submit, open a popup that shows a duplicate warning, so you have to submit two times.

Or maybe just add both Options...

In order to achieve this feature, the route in the backend has to be implemented first. Check vocascan/vocascan-server#91 for status

ShivaRapolu01 commented 2 years ago

So if the same word(string) is typed in both foreign word and translation then a warning should be shown right? If this is it then I would like to contribute to this issue. Please assign me this

noctera commented 2 years ago

So if the same word(string) is typed in both foreign word and translation then a warning should be shown right? If this is it then I would like to contribute to this issue. Please assign me this

Checking the foreign word should be enough, as translations might defer from vocab sheet to vocab sheet (for this I would like to hear the opinion of @luwol03 )

You should add a warning below the input field. We already heave errors built into it, so maybe you can just adabt/rewrite it to support warnings too.

On top of that add a confirmation dialogue that pops up using the submit button.

Currently there is one downside with this Issue, as the Routes are not yet implemented. The status of progress is referenced in this issue here. But I think this shouldn't be a problem, as it can be simulated with test Objects temporarily to check if it is working. Please think of changing it to the "Api call" when creating the PR

noctera commented 2 years ago

@ShivaRapolu01 I have just written with @luwol03 and we will check foreign word and translation. So we have different warnings for different duplicate types (all information will come from the server). I will update the issue for the new route tomorrow, so you can see the expected parameters and responses (vocascan/vocascan-server#91)