booleanbites / houzi-support

Support forum repo for issues & bugs reported for Houzi Flutter App.
4 stars 0 forks source link

It needs a lot of polish #8

Closed roccinio closed 10 months ago

roccinio commented 1 year ago

Hello,

There is a ton of issues that i have to either modify the code or try to figure out ways to bypass them and this is not sustainable in the long run. I have tried to contact you through email to no avail so i am going public here in order for you to see them and fix them.

Social logins do not go through your refactored code to use the json so they are still there even if you disable them in the json. I had to manually remove them.

The top bar widget is broken. Especially the id search does nothing and ends up in a blank widget. I have explained this in detail in one of my emails. The whole idea behind the location and the sort functionality in the top bar is non functional

OTP. How do we set it up? I have set up many firebase otp's in the past for https but it would be nice if you added that in the manual so i wont have to go reading all the docs about the flutter package.

There are overflows in some widgets in some mobile configurations. I will explain in detail when the time comes.

There are more but these i remember from the top of my head.

Regards

AdilSoomro commented 1 year ago

Hello, sorry you’re having trouble.

Regarding configuration.json changes, these will take effect during testing in the case:

we kept it this way by design, because these configurations can be delivered from web plugin as well. So to indicate latest configuration and override the existing, we rely on version code.

ID Search Issue: I might have missed the email, which rarely happens, please forward again.

Regarding phone signup, all configuration that needs to be done are on firebase and google cloud. There’s absolutely nothing need to be done in app, except replacing firebase json or plist file in relevant projects. Read docs here: https://houzi-docs.booleanbites.com/tools/sign_in_phone/

Additionally I’m available for a support session via TeamViewer. Let me know what is best suited time for you.

AdilSoomro commented 1 year ago

Hello,

Thank you for such dedication and attention to details for the app. In our session these were the things that you pointed out and we'll keep track of the progress here:

❌ Social login buttons not being controlled via config. ❌ ID search not showing error when input wrong id. ❌ Issue with json not loading in Houzi Builder. ❌ City not showing effect on latest properties. ❌ Switch (buy, rent) no visual effect. ❌ Drop the number restriction in ID search.

roccinio commented 1 year ago

Happy to help. You seem like a cool dude.

You need to also fix the show all (tabbed theme) in featured properties to work.

Add the search Id in the search page and ideally on top( first one)

If you have the Id search in carrousel top bar then it should be in the rest as well. Consistency is key for a nice UI.

Make modular all the login info like demo, add property and stuff.

I know how to get rid of them but a normal Joe what would do if he wants to publish his app without the demo for example?

And a technical question . I saw that you are correctly using the cached image plugin to store locally the assets. What image are you fetching from the WordPress media folder? The original or the optimized if I use for example shortimage (compression) plugin?

Regards


From: Adil Soomro @.> Sent: Tuesday, November 8, 2022 10:37:40 PM To: AdilSoomro/houzi-support @.> Cc: roccinio @.>; Author @.> Subject: Re: [AdilSoomro/houzi-support] It needs a lot of polish (Issue #8)

Hello,

Thank you for such dedication and attention to details for the app. In our session these were the things that you pointed out and we'll keep track of the progress here:

❌ Social login buttons not being controlled via config. ❌ ID search not showing error when input wrong id. ❌ Issue with json not loading in Houzi Builder. ❌ City not showing effect on latest properties. ❌ Switch (buy, rent) no visual effect. ❌ Drop the number restriction in ID search.

— Reply to this email directly, view it on GitHubhttps://github.com/AdilSoomro/houzi-support/issues/8#issuecomment-1307796802, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACB6XZQM4UBZMKTT4FXBHQ3WHK2ZJANCNFSM6AAAAAAR2RQE5Q. You are receiving this because you authored the thread.Message ID: @.***>

AdilSoomro commented 1 year ago

Hello,

we're using wordpress methods in plugin to fetch wordpress media images, for example:

get_the_post_thumbnail_url( get_the_ID(), 'houzez-property-thumb-image' );
wp_get_attachment_url($imgID);
wp_get_attachment_image_src($imgID, 'thumbnail', true )[0];
AdilSoomro commented 10 months ago

Most of things in these suggestions were implemented.

✅ Social login buttons not being controlled via config. ✅ ID search not showing error when input wrong id. ✅ Issue with json not loading in Houzi Builder. ✅ City not showing effect on latest properties. ✅ Switch (buy, rent) no visual effect. ✅ Drop the number restriction in ID search.

and from the suggestions:

✅ search by id is a configurable field in filters via Houzi Builder. ✅ all home top right button can be customized with hooks. Doc ✅ The left menu is a configurable, customizable menu.

Marking this as closed. Please open new issue when required.