Skip to content

Conversation

@AA-Turner
Copy link
Contributor

@AA-Turner AA-Turner commented Jul 16, 2025

The system doesn't permit multiple 'contacts' as far as I can tell, so I've added lines in the body text for 'contacts' -- happy to add anyone else who wants to be listed.

A

🖼️ Preview available 🖼️ : https://ep-website--1454.org.readthedocs.build/

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Please can you check the RTD build failure?

For example

pythonLevel**: **pythonLevel: Invalid enum value. Expected 'Any' | 'Beginner' | 'Intermediate' | 'Advanced', received 'Intermediate [Core] or Any [Documentation]'

@AA-Turner
Copy link
Contributor Author

Sure but this seems like a restriction that should be relaxed? We need to express that beginners are fine for documentation work, but intermediate+ for core.

For an immediate fix, perhaps 'Any', and then put the actual requirements in body text rather than metadata? Thoughts @hugovk? Who's best to ask re: making 'experience' a free-text field?

A

@nikoshell
Copy link
Contributor

Sure but this seems like a restriction that should be relaxed? We need to express that beginners are fine for documentation work, but intermediate+ for core.

For an immediate fix, perhaps 'Any', and then put the actual requirements in body text rather than metadata? Thoughts @hugovk? Who's best to ask re: making 'experience' a free-text field?

A

We use pythonLevel as filter field, please use description for detailed requirements.

@AA-Turner
Copy link
Contributor Author

@nikoshell does this work?

A

@AA-Turner
Copy link
Contributor Author

I've opened #1456 with a filtering problem I noticed.

A

@hugovk
Copy link
Member

hugovk commented Jul 16, 2025

Looking at the preview: https://ep-website--1454.org.readthedocs.build/sprints/#heading-11

This creates three H3 headers, the latter two at a larger font, all of which appear in the side menu:

image

Mid-conference, a quick fix is to list them as different sprints?

@AA-Turner AA-Turner requested a review from hugovk July 16, 2025 20:50
@AA-Turner
Copy link
Contributor Author

H5 fixes the sidebar issue, and the font is now the same size (though the latter two headings are in boldface). Is this OK, or should we abandon & split the pages?

A

@nikoshell nikoshell self-assigned this Jul 16, 2025
@nikoshell nikoshell self-requested a review July 16, 2025 22:14
@nikoshell
Copy link
Contributor

H5 fixes the sidebar issue, and the font is now the same size (though the latter two headings are in boldface). Is this OK, or should we abandon & split the pages?

A

There is many ways to go out of boundaries, we try to handle that, thanks for reporting.
In your case I suggest to split.

fyi: PR #1458

@AA-Turner
Copy link
Contributor Author

@nikoshell thanks, I've now split.

A

@AA-Turner AA-Turner changed the title Expand the Python sprint to cover documentation Add a Python documentation sprint topic Jul 17, 2025
AA-Turner and others added 3 commits July 17, 2025 07:53
Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Hugo van Kemenade <[email protected]>
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Thank you! 📚

@hugovk hugovk merged commit 2f057e4 into EuroPython:ep2025 Jul 17, 2025
3 checks passed
@AA-Turner AA-Turner deleted the sprints/python-docs branch July 17, 2025 07:48
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.

3 participants