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

Add local Django community page #1371

Merged
merged 6 commits into from
Nov 25, 2023

Conversation

datur
Copy link
Contributor

@datur datur commented Jun 5, 2023

Issue: #1239

This change adds a new Local Django Communities page to the community area of the website. The aim is to list Local Django Communities around the world.

Community Page:
CleanShot 2023-06-12 at 11 47 52@2x

Local Django Community page:
CleanShot 2023-06-05 at 10 24 04@2x

@datur datur marked this pull request as draft June 5, 2023 09:28
@datur datur marked this pull request as ready for review June 5, 2023 09:38
@cgl cgl requested a review from a team June 7, 2023 20:24
@camilonova
Copy link
Contributor

Please resolve the conflicts

@datur datur force-pushed the add-local-django-community-page branch from 1478bb6 to 7bc0b7b Compare June 12, 2023 10:47
@datur
Copy link
Contributor Author

datur commented Jun 12, 2023

Please resolve the conflicts

Conflicts resolved now 😄

@pauloxnet
Copy link
Member

Do we need an additional package and SVG from Material icons to add this feature to the website?

@sabderemane
Copy link
Member

Do we need an additional package and SVG from Material icons to add this feature to the website?

@pauloxnet I don't think so according what I've seen on the changes, I'll review the PR to double check

@sabderemane sabderemane self-requested a review November 2, 2023 10:37
datur and others added 5 commits November 25, 2023 01:31
This django package provides the equivalent of a choice field containing a list of all the countries in the world.

This will be used when adding a Local Django Community to the new section of the community page.
This model facilitates the listing of local django communities from around the world. This will be listed on a page within the community section of the website.
This change adds the local django communities page to the communities section of djangoproject.com
@sabderemane sabderemane force-pushed the add-local-django-community-page branch from 7bc0b7b to 4aa4c97 Compare November 25, 2023 01:19
Copy link
Member

@sabderemane sabderemane left a comment

Choose a reason for hiding this comment

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

Sorry for the late review. Thank you @datur for your contribution 🚀

I've made some small changes to improve accessibility and added the case of if there's no data.

Welcome on board ⛵

@sabderemane sabderemane merged commit dfa7429 into django:main Nov 25, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants