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

Fix issue with downloads on the usage dashboard #2866

Merged
merged 2 commits into from
Apr 13, 2021
Merged

Conversation

briri
Copy link
Contributor

@briri briri commented Apr 12, 2021

Fixes https://github.com/DigitalCurationCentre/DMPonline-Service/issues/436

We changed around the organization of the Stat models and this one file was missed. We abstracted a bunch of redundant methods from the various StatX models and added them to an abstract Stat model which these now inherit from.

@raycarrick-ed raycarrick-ed merged commit 5ee7b07 into master Apr 13, 2021
@raycarrick-ed raycarrick-ed deleted the bug-usage branch April 13, 2021 09:04
@magdalenadrafiova
Copy link

this threw the following message:
Uploading Screenshot 2021-04-13 at 11.15.07.png…

@magdalenadrafiova
Copy link

@magdalenadrafiova to retest and if fixed close

@magdalenadrafiova
Copy link

@briri @raycarrick still throws an error message :(
I am using Mac > Google Chrome > and I logged in https://dmponline.manchester.ac.uk/ and https://dmponline.dcc.ac.uk/ and got the following message:
Screenshot 2021-04-15 at 11 30 51
Screenshot 2021-04-15 at 11 29 15

@raycarrick-ed raycarrick-ed self-assigned this Apr 20, 2021
@raycarrick-ed
Copy link
Contributor

@pherterich @magdalenadrafiova
once more - lets test and see if it's fixed. Should have been fixed as part of the restart tonight.

@magdalenadrafiova
Copy link

magdalenadrafiova commented Apr 21, 2021

works thank you!

portagenetwork pushed a commit to portagenetwork/roadmap that referenced this pull request Feb 24, 2022
Fix issue with downloads on the usage dashboard
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