search

semantic-systems / nfdi-search-engine

A lightweight, KG-driven search engine over different endpoints and APIs
https://nfdi-search.nliwod.org/
MIT License
5 stars 3 forks source link

Enhancements and Feature Integrations: Dynamic Org Details, Maps, Sub-Orgs, Sub-Projects, Show-more, show-less and Error-Template page #186

Closed Najeeb-Shams closed 7 months ago

Najeeb-Shams commented 8 months ago

Hello dear colleagues, I'm writing to request a code review and merge for the following enhancements and feature integrations:

Dynamic Organization Details Page: I have added a dynamic details page for organizations. Now, you can view comprehensive information about organizations, including projects, sub-organizations, graphical maps, and general error template details.

Map Services Integration: I have integrated map services to display the location of the organization. This addition enhances the user experience by providing geographical context.

Sub-Organizations Integration: The dynamic organization-details page now includes information about sub-organizations. You can easily explore the hierarchy of related organizations.

Sub-Projects Integration: Similarly, sub-projects have been integrated into the dynamic organization-details page. This feature allows us to display a list of relevant projects associated with the organization.

"Show More" and "Show Less" functionalities have been added to the dynamic-org page. By default, this page displays 5 results. Using "Show More" will reveal all the results, while "Show Less" will hide the additional results, leaving only the default 5.

General Error Page Template: A general error page template has been added to improve the error-handling process, ensuring a more user-friendly experience in case of unexpected issues.

Please review the code, functionality, and user experience to ensure that everything aligns with our project goals. Thank you,

abdullah-rana commented 7 months ago

In org_details method in gepris.py, organization has been initialized from place class (line 232). Is it intentional? Shouldn't the organization be an instance of organization class and then the location details or anything related to geo information can be an instance of place class.