Skip to content

adds app update manager#46

Draft
zimmermannubique wants to merge 2 commits intomainfrom
feature/version-management
Draft

adds app update manager#46
zimmermannubique wants to merge 2 commits intomainfrom
feature/version-management

Conversation

@zimmermannubique
Copy link
Contributor

  • Ein einfaches Feature, dass man immer wieder mal implementiert, hier mal einfach gelöst

  • Einfaches AppVersion struct: major.minor.patch

  • Im Callback ist man in der Phase wo die Migration stattfindet (schöner als State zurückgeben, wo schon neue Version als lastVersion geschrieben)

  • Falls State ein Update kriegt man noch Info, ob Major, Minor oder Patch Update. (Evtl. nicht so aussagekräftig, User kann ja von irgendwo updaten)

  • einfache Möglichkeit zum Migrieren von anderer Versionsupdatelogik mit overwriteLastVersion Funktion

@zimmermannubique zimmermannubique requested a review from a team February 9, 2023 20:21
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link

github-actions bot commented Feb 9, 2023

Unit Test Results

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 802a9ea. ± Comparison against base commit 6385290.

Copy link
Contributor

@maerki maerki left a comment

Choose a reason for hiding this comment

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

Nach SOTA-Diskussion: Wir würden gerne noch ein paar häufige Use-Cases von Migrationen ergänzen, mit der API jetzt hat man potentiell viele Flows innerhalb vom Migrate.

@@ -0,0 +1,88 @@
//
// UBUpdateMAnager.swift
Copy link
Contributor

Choose a reason for hiding this comment

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

A → a

public class UBAppUpdateManager {
// MARK: - Shared Manager

public static let shared = UBAppUpdateManager()
Copy link
Contributor

Choose a reason for hiding this comment

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

private init

@maerki maerki marked this pull request as draft October 28, 2024 06:51
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.

2 participants