Skip to content

Commit 80f1663

Browse files
authored
Merge pull request #93 from MisterGC/review-controller-updates
clay_ios: Remove extra dialog for review
2 parents fae8fe4 + 373529f commit 80f1663

File tree

1 file changed

+2
-22
lines changed

1 file changed

+2
-22
lines changed

plugins/clay_ios/AppReviewController.qml

+2-22
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ Item {
3636
property int maxPromptCount: 3
3737

3838
// Cooldown in days between two review prompts
39-
property int cooldownDays: 30
39+
property int cooldownDays: 14
4040

4141
// Minimum active time the user has to spend before the
4242
// first request/between requests
@@ -73,7 +73,7 @@ Item {
7373
// conditions are met
7474
function showReviewPromptOnDemand() {
7575
if (reviewPromptConditionsMet()) {
76-
feedbackDialog.open();
76+
_requestReview();
7777
}
7878
}
7979

@@ -104,11 +104,6 @@ Item {
104104
console.warn("Review requests are only supported on iOS.");
105105
}
106106

107-
function handleNegativeFeedback() {
108-
// TODO: Do we need a special treatment
109-
// for rejected review requests?
110-
}
111-
112107
function _daysSince(dateString) {
113108
let date = new Date(dateString);
114109
let now = new Date();
@@ -140,19 +135,4 @@ Item {
140135
}
141136
}
142137
}
143-
144-
Dialog {
145-
id: feedbackDialog
146-
title: "Feedback"
147-
anchors.centerIn: parent
148-
modal: true
149-
standardButtons: Dialog.Ok | Dialog.Cancel
150-
onAccepted: _requestReview()
151-
onRejected: handleNegativeFeedback()
152-
153-
Text {
154-
id: _feedbackDialogText
155-
anchors.centerIn: parent
156-
}
157-
}
158138
}

0 commit comments

Comments
 (0)