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

Feature: Matching Tiles Challenge #425

Open
duckcommit opened this issue Feb 6, 2024 · 10 comments
Open

Feature: Matching Tiles Challenge #425

duckcommit opened this issue Feb 6, 2024 · 10 comments
Labels
enhancement New feature or request feat

Comments

@duckcommit
Copy link
Contributor

Description

New challenge to wake up!! Let's add a matching tiles game where the user needs to select the matching tiles, which will be shuffled and upside down. The user needs to match it based on memory. I would like to work on that,

Screenshots

No response

@MarkisDev MarkisDev added enhancement New feature or request feat labels Feb 9, 2024
@MarkisDev
Copy link
Collaborator

I've assigned this to you

@Gaurav-Kushwaha-1225
Copy link
Contributor

Hey @AryanSarafDev, I would like to work on this issue.

@AryanSarafDev
Copy link
Collaborator

Let's add a matching tiles game where the user needs to select the matching tiles, which will be shuffled and upside down. The user needs to match it based on memory. I would like to work on that,

@Gaurav-Kushwaha-1225 Sure you can work on it. But try implementing fancy animations for the tiles

@khush1yaaar
Copy link

hey if the Issue is not assigned I would love to add this one. I had built this game earlier -

piano.tiles.video.mp4

is this what you are talking about?!

@AryanSarafDev
Copy link
Collaborator

AryanSarafDev commented Nov 5, 2024

@khush1yaaar I guess @Gaurav-Kushwaha-1225 is working on it. And no we are not talking about piano tiles. It is a memory based tile flipping game Screenshot_2024-11-05-12-53-26-803_com.google.android.googlequicksearchbox.jpg

@Gaurav-Kushwaha-1225
Copy link
Contributor

Hello @AryanSarafDev,

I encountered an issue while working on the new feature. Here are the relevant links:

The problem I'm facing is that after implementing these changes, the home page is stuck on loading and is not displaying the alarms. Additionally, the selectedProfile in the HomeController is showing up as an empty string, which should show as Default.

I suspect this might be due to the FutureBuilder in home_view.dart not being able to load the future properly.

Could you please take a look and help me identify why this is happening?
Thanks in advance!

document_6181221167517406094.mp4

@AryanSarafDev
Copy link
Collaborator

Hello @AryanSarafDev,

I encountered an issue while working on the new feature. Here are the relevant links:

The problem I'm facing is that after implementing these changes, the home page is stuck on loading and is not displaying the alarms. Additionally, the selectedProfile in the HomeController is showing up as an empty string, which should show as Default.

I suspect this might be due to the FutureBuilder in home_view.dart not being able to load the future properly.

Could you please take a look and help me identify why this is happening? Thanks in advance!

document_6181221167517406094.mp4

@Gaurav-Kushwaha-1225 Sure will have a look at it

@khush1yaaar
Copy link

I am a proficient flutter developer, new to open source. please assign me this issue.

@AryanSarafDev
Copy link
Collaborator

Hello @AryanSarafDev,

I encountered an issue while working on the new feature. Here are the relevant links:

The problem I'm facing is that after implementing these changes, the home page is stuck on loading and is not displaying the alarms. Additionally, the selectedProfile in the HomeController is showing up as an empty string, which should show as Default.

I suspect this might be due to the FutureBuilder in home_view.dart not being able to load the future properly.

Could you please take a look and help me identify why this is happening? Thanks in advance!

document_6181221167517406094.mp4

ah I went through your code. This usually happens when the model for alarms is changed. Did you make sure you changed the model usage everywhere?

@Gaurav-Kushwaha-1225
Copy link
Contributor

Gaurav-Kushwaha-1225 commented Dec 13, 2024

ah I went through your code. This usually happens when the model for alarms is changed. Did you make sure you changed the model usage everywhere?

Hi @AryanSarafDev,

I've updated the model usage everywhere based on your earlier guidance. However, I'm encountering the same issue.

  • Like, for non-signed-in users, userModel is null, and their alarm data is being stored locally in the Isar DB, as usual.
  • Since I updated the alarm model to include new parameters, I expected the new alarms in the DB to follow the new model format.
  • However, the snapshot in home_view.dart is still returning null data from local storage.

I believe I've updated all instances where the alarm model is used, and

  • iff the new model was being used in the old format anywhere, I would have expected an error because the new model requires the new parameters i.e. isMatchTilesEnabled and tilesDifficulty.

Could you please review my changes or provide some guidance on what might be causing this behavior?
I suspect I might have overlooked something subtle or misunderstood part of the implementation.
If discussing this here feels inconvenient, I'd be happy to have a quick chat on Slack for clarity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feat
Projects
None yet
Development

No branches or pull requests

5 participants