Skip to content

new api version implemented#8

Merged
Tuni merged 20 commits intodevfrom
DevNewApiVersion
Dec 21, 2020
Merged

new api version implemented#8
Tuni merged 20 commits intodevfrom
DevNewApiVersion

Conversation

@dmytrolutsyk
Copy link
Collaborator

No description provided.

@dmytrolutsyk dmytrolutsyk added the enhancement New feature or request label Dec 7, 2020
@dmytrolutsyk dmytrolutsyk requested a review from Tuni December 7, 2020 10:11
dmytrolutsyk and others added 2 commits December 7, 2020 16:38
modified add_build_to_beta_group because it does not return a BetaGroup object
modified modify and create app_store_version functions
PATCH = 3
DELETE = 4

class AppStoreState(Enum):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je me demande si cette classe a sa place ici. Ca a aucune utilité à la lib il me semble, c'est utilisé pour notre logique à nous. T'en penses quoi ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suppose que dans notre usage, la suppression d'élément ne doit se faire à aucun moment sans l'intervention humaine. C'est pas forcément la bonne idée d'automatiser la suppression. On en n'a pas besoin. Mais dans la lib la méthode delete ne représente aucun danger, on l'appelle nulle part .

@Tuni Tuni merged commit 9b403fc into dev Dec 21, 2020
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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants