uprm-inso4117-2023-2024-s2 / semester-project-study-pet

semester-project-study-pet created by GitHub Classroom
1 stars 3 forks source link

`happiness` aspect #208

Closed CaribbeanCool closed 7 months ago

CaribbeanCool commented 7 months ago

Description

This branch it up to date and contains the happiness aspect of the Pet accessible and with data permanence, and it's connected to the Eat page and minigame.

close #169

CaribbeanCool commented 7 months ago

@PinkSylvie @irsaris Here is my task. Please make sure it runs on your end before merging.

PinkSylvie commented 7 months ago

I'll check it out ASAP

PinkSylvie commented 7 months ago

I ran it and tried it but it doesn't seem to affect the happiness and hunger in the stats screen

PinkSylvie commented 7 months ago

@CaribbeanCool

CaribbeanCool commented 7 months ago

@CaribbeanCool

Yeah, no añadi eso. lo hago ahora

PinkSylvie commented 7 months ago

I tried it out and neither the happiness nor the hunger are affected by the eat mini game in the stat screen

PinkSylvie commented 7 months ago

@CaribbeanCool

CaribbeanCool commented 7 months ago

@CaribbeanCool

Weird because when I use it on iOS, when I finish a minigame(new happiness is 133), it will show on Stats.js that I have 133% Happiness, and hunger isn't my task

PinkSylvie commented 7 months ago

I tested it on IOS. I even decreased the happiness manually from the default value it had so I could watch it increase after playing but didn't see any change in the stat screen

CaribbeanCool commented 7 months ago

@PinkSylvie image image

After a refresh:

image image

PinkSylvie commented 7 months ago

Why is happiness going above 100%? But I can see the changes now being applied onto the stat screen. I just think that if happiness is full it shouldn't get any fuller so that the pet follows the schedule that if it gets no care in 10 hours its happiness will decrease completely to 0. Allowing the happiness to go above 100% allows the user to create a buffer against that.

CaribbeanCool commented 7 months ago

@PinkSylvie Ya puse el cap al happiness.