onOffice-Web-Org / oo-wp-plugin

onOffice for WP-Websites
https://wp-plugin.onoffice.com
GNU General Public License v3.0
9 stars 9 forks source link

Implement City-Select into WP-Plugin #655

Closed fredericalpers closed 6 months ago

fredericalpers commented 12 months ago

INITIATIVE

Improve the usability of the onOffice for WP-Websites Plugin for a better user experience.

EPIC

Implement a City-Select.

USER STORY

As a real estate agent utilizing the onOffice for WP-Websites Plugin, I recognize the importance of providing a seamless and user-friendly property search experience for my customers. Currently, the city field is a simple search input, lacking the ability to display available cities where I have listed properties.

To enhance the search functionality and optimize the user experience, I propose the implementation of a feature that transforms the city field into a select dropdown. This dropdown should dynamically populate with only the cities where I have listed properties, ensuring that customers can easily select from the available options.

By implementing this feature, the onOffice for WP-Websites Plugin can significantly enhance the property search experience for customers. This improvement ensures that customers have a straightforward and efficient way to select cities based on the available properties listed on my real estate website, contributing to a more user-centric and engaging property search process.

TASKS

ort2

dai-eastgate commented 11 months ago

@fredericalpers I found a solution to resolve this issue. However, during implementation, I see a point below:

Filter available cities: The dropdown should only display cities where I currently have properties listed. This ensures that customers can only select from the citites relevant to the available properties on my website.

=> To resolve the above requirement, I will call an API to retrieve all the estates of the onOffice enterprise. However, according to the API document (https://apidoc.onoffice.de/actions/datensatz-lesen/objekte/), each time call API, the response "listlimit" is 500 records image

So, if your clients have more than 500 estates, this solution is not possible. Please let me know your opinion on the above issue. Thanks!

fredericalpers commented 11 months ago

@dai-eastgate yes unfortunately there is the estate limit, but most of our customers do not have 500 published properties. Please go ahead and try to implement your solution. Thank you :)

dai-eastgate commented 11 months ago

@dai-eastgate yes unfortunately there is the estate limit, but most of our customers do not have 500 published properties. Please go ahead and try to implement your solution. Thank you :)

Thanks. I need 5 days to implement and test it.

dai-eastgate commented 11 months ago

@fredericalpers I would like to confirm with you a point: To implement this feature, we must edit "fields.php" in the "template.dist" directory. => Old users need to update "template.dist" for this feature to work properly. Please let me know your opinion about this. Thanks!

fredericalpers commented 11 months ago

@dai-eastgate I will get back to you on this asap, let me confirm with my colleagues again :)

dai-eastgate commented 11 months ago

@dai-eastgate I will get back to you on this asap, let me confirm with my colleagues again :)

@fredericalpers Do you have any updates for this issue?

dai-eastgate commented 11 months ago

@fredericalpers Please watch my video demo and let me know your opinion about this. Thanks! https://files.fm/f/f5gngnwef8

Note: As I mentioned before "Old users need to update "template.dist" for this feature to work properly"

fredericalpers commented 11 months ago

@dai-eastgate thank you :) after finallybeing able to discuss this with my colleagues today I have the following feedback:

We are aware that our customers have to update their fields.php after the update. Therefore we want to implement a notice under the checkbox explaining it and link to our documentation for detailed explanation. I will send you the text and link next week, after I returned from my vacation.

Also we want to confirm that even when a customer does not update the fields.php the functionality of the city-field must still be given as before. (text input)

dai-eastgate commented 11 months ago

@dai-eastgate thank you :) after finallybeing able to discuss this with my colleagues today I have the following feedback:

We are aware that our customers have to update their fields.php after the update. Therefore we want to implement a notice under the checkbox explaining it and link to our documentation for detailed explanation. I will send you the text and link next week, after I returned from my vacation.

Also we want to confirm that even when a customer does not update the fields.php the functionality of the city-field must still be given as before. (text input)

@fredericalpers I got it. Thanks!

dai-eastgate commented 11 months ago

Also we want to confirm that even when a customer does not update the fields.php the functionality of the city-field must still be given as before. (text input)

@fredericalpers Yes, the feature "when a customer does not update the fields.php the functionality of the city-field must still be given as before. (text input)" works fine. Please take a look at my attached video.

https://github.com/onOffice-Web-Org/oo-wp-plugin/assets/106214469/8d38871f-721d-4b6d-9eee-31a6b4ac0740

Therefore we want to implement a notice under the checkbox explaining it and link to our documentation for detailed explanation.

By the way, I have implemented a notice below this checkbox. Please see the attached photo. ^^ image

fredericalpers commented 11 months ago

@dai-eastgate looks good to me, thank you :)

andernath commented 6 months ago

@dai-eastgate could you take a look at the unit tests? They are failing after merging this branch in master.

dai-eastgate commented 6 months ago

@andernath I will check