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

Wrong default site configuration #30

Closed
Silidrone opened this issue Jan 17, 2024 · 2 comments · Fixed by #31
Closed

Wrong default site configuration #30

Silidrone opened this issue Jan 17, 2024 · 2 comments · Fixed by #31
Assignees
Labels
bug Something isn't working

Comments

@Silidrone
Copy link

I freshly installed tutor version 17.0.0 and I suppose with the new host (local.edly.io) there is a problem in the site configuration of credentials service. I apologize if this is a problem on my end, but I highly doubt it as I freshly installed the credentials plugin and just enabled it.

When I run copy_catalog command, I get:
Screenshot from 2024-01-17 13-04-38
And inside admin, the site configuration is:
Screenshot from 2024-01-17 13-04-07
Needless to say, when I change http to http://, everything works.

tutor-credentials plugin is version 17.0.0

@DawoudSheraz DawoudSheraz added the bug Something isn't working label Jan 18, 2024
@DawoudSheraz
Copy link
Contributor

This appears to be an error in tutor-credentials init task (https://github.com/overhangio/tutor-credentials/blob/master/tutorcredentials/templates/credentials/tasks/credentials/init#L11). Comparing it with tutor-discovery (https://github.com/overhangio/tutor-discovery/blob/master/tutordiscovery/templates/discovery/tasks/discovery/init#L22), the :// are present in discovery.

@DawoudSheraz DawoudSheraz moved this from Pending Triage to Backlog in Tutor project management Jan 18, 2024
@Danyal-Faheem Danyal-Faheem self-assigned this Jan 18, 2024
@Danyal-Faheem Danyal-Faheem moved this from Backlog to In Progress in Tutor project management Jan 18, 2024
@Danyal-Faheem Danyal-Faheem moved this from In Progress to In review in Tutor project management Jan 18, 2024
@github-project-automation github-project-automation bot moved this from In review to Done in Tutor project management Jan 18, 2024
@Danyal-Faheem
Copy link
Contributor

This issue has now been fixed in #31. It will be incorporated in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

3 participants