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

Export accounting table as excel/.csv file #15

Closed
tomlebl opened this issue Oct 12, 2022 · 16 comments · Fixed by #127
Closed

Export accounting table as excel/.csv file #15

tomlebl opened this issue Oct 12, 2022 · 16 comments · Fixed by #127
Assignees
Labels
enhancement New feature or request

Comments

@tomlebl
Copy link
Contributor

tomlebl commented Oct 12, 2022

Add button that can export accounting table as an excel or .csv file

@tomlebl tomlebl added the enhancement New feature or request label Oct 12, 2022
@tomlebl tomlebl moved this to Todo in NOMAD system Oct 12, 2022
@tomlebl tomlebl moved this from Todo to Backlog in NOMAD system Nov 27, 2022
@tomlebl tomlebl moved this from Backlog to Todo in NOMAD system Jun 6, 2024
@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 10, 2024

@hunxjunedo would you fancy to have a look on this one?

@hunxjunedo
Copy link
Contributor

sure! Updating on this soon.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 10, 2024

Great. The export button should got to AccountingControls.jsx next to "Add Grant" button. It would be active only if there would be any results in the table and it would trigger download of the data in the table/state in .csv file.

@tomlebl tomlebl moved this from Todo to In Progress in NOMAD system Jul 10, 2024
@hunxjunedo
Copy link
Contributor

@tomlebl I looked into and didn't find any way to export csv in a way that supports sub-columns, as seen in the table. Should we flatten the columns ? Like Alec Cost 1, Alec Cost 2 ? Do you have a solution ?

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 22, 2024

@hunxjunedo I am sorry for the late response. I was away for a week on a family holiday in the middle of nowhere and it was really difficult to follow up work related things.
Flattening up the columns seems to be a good solution. I would suggest
Alec Exp Time Manual, Alec Exp Time Auto , Alec Cost and repeat analogously for the other instruments.

@hunxjunedo
Copy link
Contributor

hey @tomlebl I'm almost done, but facing issue trying to install a few npm packages. npm install packagename doesn't seem to work, docker complexities.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 23, 2024

Do you know what packages are giving you the troubles?

@hunxjunedo
Copy link
Contributor

I just can't install a new package to the frontend, it says NPM read error.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 23, 2024

I have never seen an error like that. It might be a problem with the package itself. What's the name of the package?

@hunxjunedo
Copy link
Contributor

image
no matter what package I try, its giving the same error. NPM is working fine with my other projects.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 23, 2024

This does not seem to be docker related. I have no idea what is going on. You will have to Google it or ask ChatGPT.
It could be related to your environment. Check this out
https://stackoverflow.com/questions/17035399/why-this-error-happend-nodejs-error-write-eio

@hunxjunedo
Copy link
Contributor

Hey nothing works, if I run the install command in main directory, it goes well. But in the frontend directory it throws the error. Will try resetting the whole env, and clone it again.

@hunxjunedo
Copy link
Contributor

Or what If I don't install, just add the name pkg.json file and push it, it should work fine on your end.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 23, 2024

Did you try to delete package-lock.json and /node_modules and then npm install again.
That usually cleared any npm i issues for me in past.

@hunxjunedo
Copy link
Contributor

hey @tomlebl I figured out it's an OS related issue as I'm using Linux, I know how to move forward, however I'm running out of time as I'm leaving for holidays, I'll be back in a week or two, sorry for the delay, will raise a PR as soon as I arrive Inshallah.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jul 25, 2024

No worries. There is no rush. I will be off again in the first week of August as well.
Enjoy your holiday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants