Skip to content
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

Capacité à uploader un fichier pour les configurations #45

Closed
PonteIneptique opened this issue Mar 22, 2018 · 2 comments · Fixed by #209
Closed

Capacité à uploader un fichier pour les configurations #45

PonteIneptique opened this issue Mar 22, 2018 · 2 comments · Fixed by #209

Comments

@PonteIneptique
Copy link
Member

PonteIneptique commented Mar 22, 2018

Idées

L'upload de fichier vers le serveur semble complètement inutile et pourrait provoquer une surchage à gérer. Par contre, on peut utiliser jQuery/Javascript pour lire le contenu d'un fichier et remplir les textarea :)

Ressources :

Actual behaviour

First situation

  1. As a normal user, when I create a corpus
  2. Click New Corpus
  3. Click on Write your own to show the control list creation as well
  4. Fields #tokens and #allowed_lemma, #allowed_POS and #allowed_morph need to received huge amount of data, most of the time they are copy pasted

Second situation

  1. As a control list administrator, when I want to edit a control list,
  2. Click on Control List > Any control list you contorl.
  3. On the home page of the control list, 3 yellow buttons are visible, they all start with rewrite
  4. On their page, a single field need to received huge amount of data, most of the time they are copy pasted

image

Expected Behaviour

Add a button so that

  1. User can click upload file
  2. The file is read client side and inserted in the text area.
Jean-Baptiste-Camps pushed a commit that referenced this issue Apr 6, 2018
* Fixed issue #44 : Added bold.

Warning : This change is breaking the former data model

* Removed flask migrate remnants

* Actually fixes issue #45 as well : moving to a proper click system

* Added tests for scripts

* Starting to implement client for importing data from CLI

* Working test builder

* Added export capacities

* Extracted corpus input conversion to avoid code duplication

* Added tests for the whole set of commands including corpus-list and corpus-dump

* Moved corpus-import to corpus-from-file
Created corpus-from-dir to easily reimport previous dump
Not tested the last one

* Finalizing wonderful at runtime generated tests for corpus-from-dir tests

* Nice link

* Quick link to last edit page
@PonteIneptique
Copy link
Member Author

There is a very simple yet working (I think) implementation of this in the following branch : a129625

@carinedengler
Copy link
Contributor

We estimate this ticket to take us 2d.

@MrGecko MrGecko linked a pull request Oct 1, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants