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

Sharepoint Server - search result tiles have no URLs by default #37

Closed
serenachou opened this issue Apr 5, 2022 · 4 comments
Closed
Assignees

Comments

@serenachou
Copy link
Contributor

Describe the bug

When searching across the Sharepoint Server source, the results displayed aren't useful because they seem to be missing URLs.

To Reproduce

Steps to reproduce the behavior:

  1. Create and set up the Sharepoint Server source
  2. Go to search page and find some Sharepoint Server source pages
  3. Open the preview on one result

Expected behavior

  • A clickable link to the source page should be present. But it's not.
  • A create date should be there and properly formatted. But it's not.

Screenshots

Screenshot 2022-04-04 at 17 09 54

Additional context

  • The clickable URLs and the dates do appear in search results when the user goes into the Source -> Schema and sets the type of the date to Date, and then saved the schema.
  • I also went to Search -> Display settings and check the URL field to be url but it was already preselected. I think what did it was just saving the explicit schema.
@nirav-elastic
Copy link
Collaborator

nirav-elastic commented Apr 6, 2022

@serenachou , This is because the base url from the endpoints is missing and hence, it redirects to localhost by default. The fix for this are already present in the outstanding PR: https://github.com/elastic/enterprise-search-sharepoint-server-connector/pull/30/files

@serenachou
Copy link
Contributor Author

copied this over to the original ticket for the team to see and retest

@prashant-elastic
Copy link

@serenachou This issue has been fixed and we have tested it. It looks fine now. Please find the screenshot attached for your reference.

FixedIssue#37

@nirav-elastic
Copy link
Collaborator

Closing this since the issues has been fixed and merged into the main branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants