Release 2#39
Merged
Merged
Conversation
This should make things much simpler instead of juggling the session. Probably still need to fix some things, though.
Simplifies a lot of other things if we just pin what study (existing or new) the form is focused on and we can't change it halfway through.
* main: Use non-deprecated logging method Allow filtering page visits by path in the admin Track API visits separately from web ones Record time measurements as WARN logs Record time measurements as DEBUG logs Measure how long it takes to track page visits
* main: Fix favicon metadata tag
* main: Allow building documentation from this branch Documentation for the "isAverage" field Return a flag about whether the bioreplicate is an "averaged" one Add redis to the github actions setup Pin more specific python version for tests Move maxminddb out to separate initializer, remove timing information Record page visits through a background job
This reverts commit b465a16.
For the API, providing an apiKey will allow people to download their own unpublished data. This is not documented yet. For the web, this will allow downloading data using g.current_user.
hariszaf
self-requested a review
June 29, 2026 10:53
Contributor
|
Approved |
hallucigenia-sparsa
approved these changes
Jun 29, 2026
hariszaf
reviewed
Jun 29, 2026
hariszaf
left a comment
hariszaf
left a comment
Member
There was a problem hiding this comment.
This is a massive PR most of which has been already reviewer, within @AndrewRadev fork.
I guess it's more than good to go, since it has been already tested in production mode.
| list and the title of a study. | ||
| """ | ||
| def __init__(self, doi): | ||
| self.doi = doi |
Member
There was a problem hiding this comment.
Consider having something like:
doi.strip('https://doi.org/') just in case ? except if you re making sure of that somewhere else
Contributor
Author
There was a problem hiding this comment.
The crossref API works fine if the DOI given is a URL starting with https://doi.org, so this works in practice. Though weirdly enough, I can't quite tell where requests is escaping it, so maybe I'll add an explicit quoting function just in case.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is intended to bring the main repository up to date with the deployed code, and there's been a lot of changes since the last one, #33. I will list high-level changes on the basis of PRs, though there's probably more changes in the "various improvements" categories: