diff options
Diffstat (limited to 'pydis_site/apps/main/views/home.py')
| -rw-r--r-- | pydis_site/apps/main/views/home.py | 35 |
1 files changed, 15 insertions, 20 deletions
diff --git a/pydis_site/apps/main/views/home.py b/pydis_site/apps/main/views/home.py index e06b3783..d9255ed2 100644 --- a/pydis_site/apps/main/views/home.py +++ b/pydis_site/apps/main/views/home.py @@ -4,15 +4,17 @@ from django.shortcuts import render from django.utils import timezone from django.views import View -from pydis_site.apps.main.models import RepoData +from pydis_site.apps.main.models import RepositoryMetadata GITHUB_API = "https://api.github.com/users/python-discord/repos" -class Home(View): +class HomeView(View): - def _get_api_data(self): + @staticmethod + def _get_api_data(): """Call the GitHub API and get information about our repos.""" + repo_dict = {repo_name: {} for repo_name in settings.HOMEPAGE_REPOS} # Fetch the data from the GitHub API @@ -20,7 +22,6 @@ class Home(View): api_data = api_data.json() # Process the API data into our dict - print(f"repo_dict = {repo_dict}") for repo in api_data: full_name = repo["full_name"] @@ -32,22 +33,18 @@ class Home(View): "forks_count": repo["forks_count"], "stargazers_count": repo["stargazers_count"], } - print(f"repo_dict after processing = {repo_dict}") return repo_dict def _get_repo_data(self): - """Build a list of RepoData objects that we can use to populate the front page.""" + """Build a list of RepositoryMetadata objects that we can use to populate the front page.""" # Try to get site data from the cache try: - repo_data = RepoData.objects.get(repo_name="python-discord/site") + repo_data = RepositoryMetadata.objects.get(repo_name="python-discord/site") - # If the data is older than 2 minutes, we should refresh it. THIS PROBABLY ALWAYS FAILS? + # If the data is older than 2 minutes, we should refresh it. if (timezone.now() - repo_data.last_updated).seconds > 120: - diff = (timezone.now() - repo_data.last_updated).seconds - print(f"okay baby, it's old! the seconds difference comes to: {diff}") - # Get new data from API api_data_container = self._get_api_data() repo_data_container = [] @@ -55,13 +52,13 @@ class Home(View): # Update or create all RepoData objects in settings.HOMEPAGE_REPOS for repo_name, api_data in api_data_container.items(): try: - repo_data = RepoData.objects.get(repo_name=repo_name) + repo_data = RepositoryMetadata.objects.get(repo_name=repo_name) repo_data.description = api_data["description"] repo_data.language = api_data["language"] repo_data.forks = api_data["forks_count"] repo_data.stargazers = api_data["stargazers_count"] - except RepoData.DoesNotExist: - repo_data = RepoData( + except RepositoryMetadata.DoesNotExist: + repo_data = RepositoryMetadata( repo_name=api_data["full_name"], description=api_data["description"], forks=api_data["forks_count"], @@ -74,10 +71,10 @@ class Home(View): # Otherwise, if the data is fresher than 2 minutes old, we should just return it. else: - return list(RepoData.objects.all()) + return RepositoryMetadata.objects.all() # If this is raised, the database has no repodata at all, we will create them all. - except RepoData.DoesNotExist: + except RepositoryMetadata.DoesNotExist: # Get new data from API api_data_container = self._get_api_data() @@ -85,7 +82,7 @@ class Home(View): # Create all the repodata records in the database. for api_data in api_data_container.values(): - repo_data = RepoData( + repo_data = RepositoryMetadata( repo_name=api_data["full_name"], description=api_data["description"], forks=api_data["forks_count"], @@ -98,9 +95,7 @@ class Home(View): return repo_data_container def get(self, request): + """Collect repo data and render the homepage view""" - # Collect the repo data repo_data = self._get_repo_data() - - # Call the GitHub API and ask it for some data return render(request, "home/index.html", {"repo_data": repo_data}) |