m0hit-kumar / Bring-The-Menu

MIT License
20 stars 45 forks source link

Added introslider and onboarding for new users #92

Closed jaivsh closed 1 year ago

jaivsh commented 1 year ago

Adding Introslider and Onboarding for new users!

Closes #89

Description

I've added onboarding and intro slider for first-time users, which would make them familiar with the usability of the app and it's features. This also includes creation of the suitable graphic images and other secondary stuff. For this, i've used the following packages from pub.dev:

intro_slider shared_preferences IsFirstRun

Screenshots

Screenshot_20230704_231703

Screenshot_20230704_231736 Screenshot_20230704_231723

Screenshot_20230704_231822

Checklist

jaivsh commented 1 year ago

Hi @m0hit-kumar can you please review this PR? Thanks!

jaivsh commented 1 year ago

Hi @m0hit-kumar I've removed all swift files, which were causing the tests to fail. Hence, can you please check them again?

jaivsh commented 1 year ago

Hi @m0hit-kumar I've recently chcked with the GSSOC team: the way in which your codeql is designed is wrong. The codeql has identified c++ and swift as your project's primary languages, which is not correct. Your project is based in flutter, so Dart turns out to be your primary language. Hence, the codeql should be set up in dart, and not in cpp or swift. This is the reason why it is failing those checks. in the PR. Hence, I'd require please review it. Thanks!

m0hit-kumar commented 1 year ago

Hi @m0hit-kumar I've recently chcked with the GSSOC team: the way in which your codeql is designed is wrong. The codeql has identified c++ and swift as your project's primary languages, which is not correct. Your project is based in flutter, so Dart turns out to be your primary language. Hence, the codeql should be set up in dart, and not in cpp or swift. This is the reason why it is failing those checks. in the PR. Hence, I'd require please review it. Thanks!

i see that, do one thing resolve the conflicts and I will merge your PR

jaivsh commented 1 year ago

@m0hit-kumar resolved conflicts, please merge!