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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

'## In The Community' in Roundup post should be at start of the line #535

Closed
claremacrae opened this issue Nov 14, 2022 · 10 comments
Closed
Assignees

Comments

@claremacrae
Copy link
Contributor

claremacrae commented Nov 14, 2022

In the roundup file 2022-11-12: Stable Diffusion, Clean Dashboards and 2022 Workflows, there is a line:

馃拵Eleanor got Lasik on Thursday so she's trying to take it easy this weekend. This issue was guest edited by [Curtis McHale](https://curtismchale.ca).## In The Community

The text ## In The Community should be on a new line. I have been unable to figure out the cause of the problem.

    I will log an issue for the `## In The Community` not being at the start of a line.

Originally posted by @claremacrae in #534 (comment)

@claremacrae
Copy link
Contributor Author

@AB1908 When/if you have time, please could you look at this?

@AB1908
Copy link
Contributor

AB1908 commented Nov 19, 2022

Okay so this looks like it's because of a callout in that specific post that is wrapped in a div. I'm guessing that this caused some parsing issues. How do we want to resolve this?

@claremacrae
Copy link
Contributor Author

Thanks!

I did do a search in the Hub for other problem cases where there was a heading not at the start of the line - and aside from a couple where there was a space before the heading, there were not others.

So I think the choices I can think of are:

  1. write clever code to handle the general case
  2. write an awful hack that converts .## to .\n\n##
  3. ask Eleanor if she would be willing to edit the article (or whatever appears in the RSS) to remove the callout or div - or whatever is causing the problem.

@AB1908
Copy link
Contributor

AB1908 commented Nov 20, 2022

I think 1 and 3 are reasonable and I'm sure Eleanor would be very open to editing it but I'm not entirely sure if those updates get pushed to the RSS. I'm also a little saddened that she can't use a callout but the less code to maintain, the better!

Should we @ her in this thread or leave a message on Discord?

@claremacrae
Copy link
Contributor Author

the less code to maintain, the better!

馃憤

Should we @ her in this thread or leave a message on Discord?

I don't have a preference. You decide!

@AB1908
Copy link
Contributor

AB1908 commented Nov 25, 2022

Hi @eleanorkonik, your call-out at the top in https://www.obsidianroundup.org/2022-11-12/ is a little tricky to work with. Would you prefer sticking to callouts or do you think that changing that to something like a heading is a decent trade-off? I personally somewhat enjoy them but it's your newsletter haha.

You'd have to be consistent with whatever you choose and I'm also curious if Ghost actually goes back and updates the RSS. If it doesn't, that'll be another problem.

Regardless, let us know what your preferred approach is and I'll try and get an appropriate fix in. Thanks!

@claremacrae
Copy link
Contributor Author

@claremacrae
Copy link
Contributor Author

For only one file, I am not sure it is worth writing special-case code.
However, if anyone would like to look at it, you're welcome, and here is what you would need to know.

The relevant code is at:

.github/scripts/update_roundup.py

  • Run it from inside .github/scripts/, and confirm first that none of the existing files in 01 - Community/Obsidian Roundup changes.
  • Then adjust the code and confirm that the only file that changes is the one with the problem heading.

@claremacrae
Copy link
Contributor Author

Eleanor has kindly converted the callout to a blockquote. I re-ran update_roundup.py on my machine and this was the diff:

image

So it is fixed.

I will close this when the new version is published.

@claremacrae
Copy link
Contributor Author

Fix now published:

image

Closing.

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

2 participants