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

use gh-issues to update conferences.json #503

Merged
merged 11 commits into from
Oct 5, 2024
Merged

Conversation

kjaymiller
Copy link
Contributor

  • update issue template to include start and end dates for conference

  • update conferences to use gh-issues instead of manual parsing

  • update conferences template to use grid and new json data

  • add conferences.json

update issue template to include start and end dates for conference

update conferences to use gh-issues instead of manual parsing

update conferences template to use grid and new json methods

add conferences.json

remove envrc
@kjaymiller
Copy link
Contributor Author

kjaymiller commented Oct 5, 2024

TODO:

  • Update Tests

update issue template to include start and end dates for conference

update conferences to use gh-issues instead of manual parsing

update conferences template to use grid and new json methods

add conferences.json
@kjaymiller kjaymiller requested a review from dragid10 October 5, 2024 19:36
@kjaymiller
Copy link
Contributor Author

@dragid10

You were the last to work on this. would you mind puttins some eyes on this change.

Copy link
Member

@dragid10 dragid10 left a comment

Choose a reason for hiding this comment

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

Awesome refactor! Just had some suggestions and a new test-case consideration

from github.Issue import Issue
from github.PaginatedList import PaginatedList
import json
import gh_issues
Copy link
Member

Choose a reason for hiding this comment

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

Me looking up the gh_issues library
image
skeptical-futurama

_conferences/__main__.py Outdated Show resolved Hide resolved
],
)
def tests_validate_issue(check_value, asserted_value):
"""tests the date validator againse yesterday value (fail) and tomorrow value (pass)"""
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"""tests the date validator againse yesterday value (fail) and tomorrow value (pass)"""
"""tests the date validator against yesterday value (fail) and tomorrow value (pass)"""

if not (valid_date := getattr(issue, date_to_check, False)):
return False
else:
return __to_conference_date(valid_date) >= datetime.date.today()
Copy link
Member

Choose a reason for hiding this comment

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

Something else that I think would make sense to check is if end_date >= start_date. Take this for example:

today_date = 10/05/2024
start_date = 10/15/2024
end_date   = 10/10/2024

This scenario will always clear the validate_issue check because the conference end date is after today's date. But it still shouldn't be considered a valid conference since the end date is before the start date. Let me know if I'm overthinking this one or not

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think maybe you're overthinking this as this validation is purely to filter out the entry from the generated json list.

These conference entries should still be validated by someone on the triage team and if there is an issue in entry, it is the triage members responsibility to detect that.

"conference_type": "in-person",
"conference_location": "NWSC International Resource Centre (IREC), Kampala, Uganda",
"summary": "The conference is a five-day event, 9-13 October. PyconUganda inherit this tradition of an annual event that brings together python enthusiasts, tech audiences and technology business in uganda and around the world.",
"speaking": null
Copy link
Member

Choose a reason for hiding this comment

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

I'm curious, what will the format of the speaking field look like? Would it be a list of json objects?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently it is a string of entries...

I was thinking of a GH-Issue feature that would that would convert a list to a python list.

with events.md being a .md file. It should render properly in the website.

@dragid10
Copy link
Member

dragid10 commented Oct 5, 2024

Well I'm good on these changes (though I don't have write access, so my approval means nothing)

@kjaymiller
Copy link
Contributor Author

@dragid10 - updated the approval limit for the triage team

@kjaymiller kjaymiller merged commit 76da3a9 into gh-pages Oct 5, 2024
2 checks passed
@kjaymiller kjaymiller deleted the update_conferences branch October 5, 2024 23:25
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