This repository has been archived by the owner on Jun 21, 2022. It is now read-only.
refactor: use storages instead of settings storage #56
Conversation
This file contains 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
Closes #42 - simplified and fixed updating storage scenario in ApiRepoUpdateSlice. On dashboard we always work with whole yaml repo settings, so there is no need to update section by section, we can simply rewrite the whole file after checking that required fields (type and storage) are present in the new version.
Added description for several repositories. Issue: artipie/artipie#317 PR: #44
Handle errors in `ApiRepoUpdateSlice` (#45) Closes: artipie/artipie#320
Added anaconda repository option to dashboard and configuration description. Ticket: artipie/artipie#317
Clarified API reference link name. Closes: #47
Part of artipie/artipie#965 Added rq line to get users in FromRqLine, corresponding test and updated http and asto.
Add slice to delete a repo. Added disabled test for this slice as it is necessary to implement delete operation in FakeConfigFile class. Ticket: artipie/artipie#321
Added slice ApiRepoPostRtSlice for routing post requests by parsing body content. This class will be used in artipie module. Test will be added in next PR. Ticket: artipie/artipie#321
* Bumped ppom from 0.5.1 to 1.1.0 * Bumped asto from v1.9.0 to v1.10.0 * Bumped http-client from 0.3.2 to 0.3.6 * Bumped vert-server from 0.4 to 0.5
Extract class for parsing body content for receiving some content. Close: #50
Enable and a bit extend test for ApiRepoDeleteSlice and add test for ApiRepoPostRtSlice. Ref: artipie/artipie#321
genryxy
requested review from
a team and
olenagerasimova
and removed request for
a team
olenagerasimova
approved these changes
Dec 21, 2021
|
@g4s8 could you please check/merge |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Part of artipie/artipie#321
Repository has storages which could differ from storage with repository settings therefore
Storageswas used to get this storage form Artipie module