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

Adding src/de/2025-01/04/teacher-questions.md and previous #3155

Closed
wants to merge 58 commits into from

Conversation

toak
Copy link
Contributor

@toak toak commented Oct 18, 2024

for adult lesson de/2024-04
from wiki.sabbatschule.at

Pull Request check-list

Please make sure to review and check all of these items:

  • Have you checked for semicolons in the titles?

Copy link
Member

@VitalikL VitalikL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@toak Unfortunately we can no longer accept improperly formatted files, please clean it up from using non-markdown format. The changes requested look like they are auto-generated

@@ -9,6 +9,9 @@ date: 18/10/2024

- Was in diesem Abschnitt könnt ihr logisch nicht nachvollziehen?

```{=html}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@toak Unfortunately we can no longer accept improperly formatted files, please clean it up from using non-markdown format.

*The message of John: here is your king!: with study guide*, The Bible
Speaks Today (Leicester, England; Downers Grove, IL: InterVarsity Press,
1993), 4

Weitere Fragen oder Ideen? - du bist herzlich zur Mitarbeit eingeladen: https://wiki.sabbatschule.at
Weitere Fragen oder Ideen? - du bist herzlich zur Mitarbeit eingeladen:
<https://wiki.sabbatschule.at>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Links should also be properly formatted

@VitalikL
Copy link
Member

@toak I spent some time to clean up almost all previous teacher-questions - you can take a look, but even then the changes I made are not ideal. Ideally we would like clean Markdown format, because we are preparing big upgrade to our API and the app to avoid HTML completely.

@toak
Copy link
Contributor Author

toak commented Oct 18, 2024

Thanks for the heads-up, I'll update the routine to remove HTML

@toak
Copy link
Contributor Author

toak commented Oct 18, 2024

The source of the content is wiki.sabbatschule.at
I have changed the converter now to be markdown_strict without HTML: (new commits)
pandoc -f mediawiki -t markdown_strict-raw_html

Let me know what else you want to have improved @VitalikL

@toak toak requested a review from VitalikL October 18, 2024 16:27
@toak
Copy link
Contributor Author

toak commented Oct 21, 2024

@VitalikL I'll fix the issues with the converter in the next days

@toak
Copy link
Contributor Author

toak commented Nov 1, 2024

@VitalikL I've added some additional rules to the conversation process, so the current files should be clean. If you notice some other issues, please comment on the PR, so I can fix the converter.

@toak toak requested a review from VitalikL November 2, 2024 22:01
@toak
Copy link
Contributor Author

toak commented Nov 2, 2024

@VitalikL everything should now be ready for the pull

toak added 20 commits November 22, 2024 00:00
@toak
Copy link
Contributor Author

toak commented Dec 9, 2024

@VitalikL do you want me to change anything before you can merge? I regularly keep things up to date to avoid merge requests
I already got the questions from users when the questions will be available.

@toak toak closed this Dec 15, 2024
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

Successfully merging this pull request may close these issues.

2 participants