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

On exit, does not prompt to save unsaved edits #851

Closed
sizezero opened this issue Oct 11, 2018 · 6 comments
Closed

On exit, does not prompt to save unsaved edits #851

sizezero opened this issue Oct 11, 2018 · 6 comments
Labels
env:Linux This issue is related to running Data Curator on Linux fn:Close-Quit status:Requires-replication This issue needs replication support This issue is a candidate to complete under the support agreement

Comments

@sizezero
Copy link

  • run datacurator
  • open a CSV file
  • edit a cell, hit enter to submit cell changes
  • exit datacurator
  • edits are silently discarded; most/all editors prompt the user to save unsaved edits
@ghost
Copy link

ghost commented Oct 11, 2018

Hi @sizezero
Might be different behaviour you're seeing on Linux (or confirm that you're using the release, not the dev build), but using the release artifact, you should see:

screen shot 2018-10-12 at 9 47 49 am

@sizezero
Copy link
Author

I'm not sure if I performed a release or a dev build. I thought there were no linux releases available for download. Here's what I did:

  • git clone https://github.com/ODIQueensland/data-curator.git
  • yarn run dev
  • yarn run build
    • this above command failed but maybe it did a partial build?
  • .electron-vue/build.js && yarn run electron-builder --linux AppImage
  • was able to run build/linux-unpacked/datacurator from the command line

Is there a different way to build that makes a "release build"?

@ghost
Copy link

ghost commented Oct 19, 2018

Hi @sizezero
I've started looking at improving our release/ci process - I might be able to put in more support for linux here as I can also see others keen for this integration: #830 #247 . Just to draw a line for now, I'll start with AppImage and test on Ubuntu 16x LTS. From here I can look at what happens with the linux build artifact.

@ghost ghost added the status:In-progress We're working on this right now label Oct 19, 2018
@sizezero
Copy link
Author

Thanks!

Let me know if I need to build the production image (and how) or if it will a be available for download.

@ghost ghost added support This issue is a candidate to complete under the support agreement status:On-hold We've stopped working on this for the moment and removed status:In-progress We're working on this right now labels Jan 31, 2019
@ghost ghost added status:Requires-replication This issue needs replication f:Feature-request This issue is a request for a new feature priority:Low and removed f:Feature-request This issue is a request for a new feature labels May 29, 2019
@markwheels markwheels added priority:11 Priority 11 and removed priority:Low labels May 30, 2019
@Stephen-Gates Stephen-Gates added this to the Support release milestone May 30, 2019
@markwheels markwheels removed priority:11 Priority 11 labels May 31, 2019
@Stephen-Gates Stephen-Gates added the env:Linux This issue is related to running Data Curator on Linux label Jun 1, 2019
@ghost ghost removed the status:On-hold We've stopped working on this for the moment label Jun 7, 2019
@Stephen-Gates
Copy link
Contributor

See also #68 for an issue about potentially losing unsaved work

@ghost
Copy link

ghost commented Jul 5, 2019

Hi

Unless I've misunderstood the issue, closing this one off as my linux build shows closing message ok.
image
CI includes all tests for linux and while it isn't available for download from releases, can be built simply enough (for AppImage), by running yarn build

@ghost ghost closed this as completed Jul 5, 2019
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
env:Linux This issue is related to running Data Curator on Linux fn:Close-Quit status:Requires-replication This issue needs replication support This issue is a candidate to complete under the support agreement
Projects
None yet
Development

No branches or pull requests

3 participants