SUBMISSION-79: Initial Use of Author's Work (was License) UI updates. #30
Open
DavidLFielding wants to merge 149 commits intomasterfrom
Open
SUBMISSION-79: Initial Use of Author's Work (was License) UI updates. #30DavidLFielding wants to merge 149 commits intomasterfrom
DavidLFielding wants to merge 149 commits intomasterfrom
Conversation
Minor changes to eliminate errors and allow additional upload MIME types.
…ory instead of arxiv-submission-ui repository.
Update links on manage submissions page to point to submit-ce repository.
Add emulator dependency to README
ARXIVCE-4072: New Review Files step (placeholder)
…xed. [SUBMISSION-20] David
SUBMISSION-20: Fix failure to progress from Submission Agreement step.
- Added `get_source_file_info` and `store_source_file` to `SubmissionFileStore` - Implemented `get_source_file_info` and `store_source_file` in `GsFileStore` - Rename `Upload` to `Workspace` - add `yarl` for `URL` - Changes `Workspace` `FileError` and `FileStatus` to pydantic instead of `NamedTuple` - Added `crc32c`, `url` and `is_versioned` to - In `FileStatus` renamed `size` to `bytes` for explicit units - In `FileStatus` renamed `file_type` to `content_type`
- Added `AddFiles`, `RemoveFiles`, and `RemoveAllFiles` events to `submit_ce/api/domain/event/file.py`. These inherit from `EventWithSideEffect` and interact with the `SubmitApi` - Added unit tests - Extracted the `SubmitFile` Protocol into a dedicated `types.py` file to resolve a circular import - Unified the `SubmissionFileStore` return types, updating `get_source_file` to return an `arxiv.files.FileObj` - Added missing abstract methods to the `SubmissionFileStore` interface. - Implemented previously missing methods in the `GsFileStore` - Added stub methods to `LegacyFileStore` to satisfy the updated abstract base class
- Changes to github workflow to standardize coverage config with pyproject - Adds test.sh to run tests similar to GHA
also fixes some ruff errors, moves
This is to fix circular import problems. The domain objects are no longer under submit_ce.api so they can successfully import and use api objects for typing. It is safe to use them just for typing since no functions are called and no classes instanciated. This is done with `annotations` and `TYPE_CHECKING` conditions. - Fixes ruff errors - Tests at get past import errors
I often don't want it run.
Tests all pass No linting errors More work on moving domain to its own packages
Also removes dome validation from submit_ce/domain/annotation.py since it is now handled by pydantic
File store operations, refactor and ruff fixes
Submission 18: Updates from latest mock ups (v5)
Renames compile service file
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
4 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 includes the initial changes to the Use of Author's Work page based on version 5 of the mockups.
This includes a license comparison modal. There are a few styling and background shading issues to fix.
Backend work remains to save license (and add licenses to the DB).
I will tag Shamsi on this work after I fix the styling issues.