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

fix: #220 UI Redesign - Mobile - Terms & Condition page #225

Open
wants to merge 5 commits into
base: ui_redesign
Choose a base branch
from

Conversation

tanmoy27112000
Copy link

- What I did
If this is a bug fix, I Closed #220 . Added the terms and condition button for the settings page

- How I did it

I created a separate file for setting page button and made it such that all button can use the same widget.

- How to verify it

Just add the following code
SettingPageButton( iconPath: ImageConstants.termsAndConditionsIcon, title: "Terms & Conditions", )

to get the widget

- Description for the changelog

Added setting page button widget

@tanmoy27112000 tanmoy27112000 changed the title Added settings page Terms and Condition Button fix: #220 UI Redesign - Mobile - Terms & Condition page Oct 2, 2022
@tanmoy27112000
Copy link
Author

@ksanty please review this

@tinashe404
Copy link
Member

@tanmoy27112000 Please could you make it so that the widget appears when someone clicks Terms & Conditions in the menu?

@tanmoy27112000
Copy link
Author

@tinashe404 sure

@tanmoy27112000
Copy link
Author

@tinashe404 you can click on the terms and condition button to see the widget

@tinashe404
Copy link
Member

@tanmoy27112000 Clicking terms and conditions opens up a page with the terms and conditions button but the button isn't doing anything.

@tanmoy27112000
Copy link
Author

@tinashe404 added onTap method
please check now.

@tanmoy27112000
Copy link
Author

@tinashe404 the Hacktoberfest website is showing the repo as non participating. since Hacktoberfest tag is not added. can that be done

@cpswan
Copy link
Member

cpswan commented Oct 8, 2022

I've added the hacktoberfest topic (and some others) to this repo

@tanmoy27112000
Copy link
Author

@tinashe404 please check this PR . Conflicts resolved

@tanmoy27112000
Copy link
Author

on tap method takes to the terms web screen @tinashe404 please check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants