anusii / solidpod

MIT License
1 stars 2 forks source link

Zy/125 review structure #219

Closed zheyxu closed 3 months ago

zheyxu commented 4 months ago

Pull Request Details

What issue does this PR address

Checklist

Complete the check-list below to ensure your branch is ready for PR.

Flutter Style Guide: https://survivor.togaware.com/gnulinux/flutter-style.html

Finalising

Once PR discussion is complete and 2 reviewers have approved:

zheyxu commented 4 months ago

DO NOT MERGE - a misunderstanding - we keep the CONTINUE button in SOLIDPOD but specify required: true in KEYPOD.

Get it. The new implementation is that keep the CONTINUE button in SOLIDPOD. If specify required: true, the CONTINUE button is dropped. If specify required: false, the CONTINUE button is kept.

zheyxu commented 3 months ago

Hi @gjwgit, make changes according to your request.