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

feature: keep prediction in UI #285

Merged
merged 10 commits into from
Jun 6, 2020

Conversation

alex-krasn
Copy link
Contributor

We're keeping predictionPage mounted until switching to another project, then remount it.

@alex-krasn alex-krasn changed the title feature: we're keepin prediction in UI now feature: we're keepin prediction in UI May 29, 2020
@alex-krasn alex-krasn changed the title feature: we're keepin prediction in UI feature: keepin prediction in UI May 29, 2020
@alex-krasn alex-krasn changed the title feature: keepin prediction in UI feature: keep prediction in UI May 29, 2020
tianxiangs
tianxiangs previously approved these changes May 29, 2020
Copy link
Contributor

@stew-ro stew-ro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
Getting error with project security token. Prevents opening app, even "/"

@alex-krasn alex-krasn force-pushed the alex-krasn/feature-keep-prediction-in-UI---#204 branch from 3d57606 to cb7ffd4 Compare June 5, 2020 18:32
@alex-krasn alex-krasn merged commit dad98b9 into master Jun 6, 2020
@alex-krasn alex-krasn deleted the alex-krasn/feature-keep-prediction-in-UI---#204 branch June 6, 2020 01:26
alex-krasn added a commit that referenced this pull request Jun 25, 2020
* feature: shortcuts icon toolbar (#288)

* fix: fix mime check bug for jpeg/jpg and tiff (#291)

* doc: update version to 2.0-2760166

* Update CHANGELOG.md

* feat: Add better error message for CORS  (#289)

* feat: refactor CORS error message

* refactor: remove unuse import and console

* refactor: fix Spanish message and add multilineMessage function in Alert

* refactor: change number to bullet points

* add div when return multiline message

* style:remove extra space

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>

* fix: check ocrread results before getting image extent (#296)

* feat: enable download analyze script (#304)

* migrating from npm to yarn (#305)

* migrating from npm to yarn

* added Typescript

* updated azure-pipeliens.yml

* style:  deletes run in scripts

* Update CHANGELOG.md

* migrating to Fluent UI (#306)

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>

* feat: add description - how to delete info (#292)

* feat: add description - how to delete info

* style: new line

* style: removes extra line

* fix: correction of text

* Update CHANGELOG.md

* Update CHANGELOG.md

* Update statusBar.tsx

* refactor: use yarn in dockerfile (#309)

* refactor: use yarn in dockerfile

* refactor: upgrade node to 12.x

* feature: keep prediction in UI (#285)

* feature: we're keepin prediction in UI now

* feat: add protection for additional hidden elements for skipButton

* bugfix: - delete forgoten changes

* rerender on train model change + refactor

* fix: forgotten log , and comment

* fix: now rerender when setting changes and on new project

* refactor: delete unnecessary helper file

* fix: add  this.props.project check

Co-authored-by: stew-ro <60453211+stew-ro@users.noreply.github.com>

* fix: fix spelling correction for string match (#318)

* fix: capitalize python in analyze page (#320)

* fix: capitalize python in analyze page

* Alex krasn/docs update readme keyboard shortcuts (#313)

* fix:  deletes trailing whitespaces

* docs: added icon and updated description

* refactor: rephrasing and icon on new line

* feat: enable re-run OCR (#297)

* feat: enable re-run OCR (draft)

* add rerun OCR for single file functionality

* added tooltip

* add icon and changed position

* added - rerun OCR for all documents in project

* fix: remove idents

* feat: enable re-run OCR (draft)

* add rerun OCR for single file functionality

* added tooltip

* add icon and changed position

* added - rerun OCR for all documents in project

* refactor: delete redundant local state and rename

* fix: indent

* style: change submenu text

* docs: added manual test runbook

* style: idents

* refactor: rename dropdown class name

* renaming: re-run.. to run, files to documents..

* fix: style and grammar

* fix: run on not visited

* docs: new Test Runbook revision

* style: fix wording and spacing

* refactor: use anonymous wrapper function for onclick

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>
Co-authored-by: Robert Stewart (eXcell <v-stewro@microsoft.com>

* feature: localization for canvas commandbar items #319 (#323)

* feat: localization all canvas commandbar items

* fix: labe in Layers - has to be plural

* style: - extra line

* fix: quick reordering tags (#322)

* fix: added debounce for quick reordering tags

* docs: added use case

* refactor: incresed dedounce interval so its not affect when reordering slowly

* decrease debounce time to 3000ms

* style: - indent

* add: show prediction component for both (receipts, predict) routes

* fixing icons + add alternative receipts icon

* + badge "Preview"

* Update README.md

* fix: error on copy APIKey undefined

* increase visibility of higlied values, separation valueNames and values ,misc styles fixes

* disable sidebar links

* fix: sigle quotes -> double

* fix: trailing whitespaces  + spelling "separator"

* redirect to receipt page + now showing python script download

* hide disabled links

* deletes prepended currency units

* fix: accessibility

* misc: styling and accessibility

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>
Co-authored-by: tianxiang <60452964+tianxiangs@users.noreply.github.com>
Co-authored-by: stew-ro <60453211+stew-ro@users.noreply.github.com>
Co-authored-by: Robert Stewart (eXcell <v-stewro@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants