Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Flow to accept terms of Integration manager #2621

Closed
wants to merge 6 commits into from
Closed

Conversation

manuroe
Copy link
Member

@manuroe manuroe commented Aug 2, 2019

Part of #2600.

This is a WIP.
There is no SDL PR yet.

There is a flow composed by 3 screens (load terms, display terms, send accepted terms) to be able to adapt to any changes in the design
This looks a bit over-engineered atm because we use only an AlertController but that can be changed next week.

I know there are 2 TODOs in this PR.

Screen Recording 2019-08-02 at 17 33 38

@manuroe manuroe requested a review from SBiOSoftWhare August 2, 2019 15:43
@manuroe
Copy link
Member Author

manuroe commented Aug 12, 2019

This PR is replaced by #2644 with a list of terms UX

@manuroe manuroe closed this Aug 12, 2019
@manuroe manuroe deleted the riot_2600 branch July 29, 2020 14:06
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