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

update timestamp for meeting agendas seem incorrect #3849

Closed
1 task done
ghwood opened this issue Apr 20, 2022 · 8 comments · Fixed by #7722
Closed
1 task done

update timestamp for meeting agendas seem incorrect #3849

ghwood opened this issue Apr 20, 2022 · 8 comments · Fixed by #7722
Assignees
Labels
accepted bug Something isn't working component: meeting/ contrib-medium Medium dev difficulty for contribution minor

Comments

@ghwood
Copy link

ghwood commented Apr 20, 2022

What happened?

The last updated timestamp seems to be too far in the past.

It also should use either the user's local timezone or UTC.

Attached is a screenshot of https://datatracker.ietf.org/meeting/114/agenda/ taken on 2022-04-20 at 9:45 in a EDT.

image

What browser(s) are you seeing the problem on?

Safari

Code of Conduct

  • I agree to follow the IETF's Code of Conduct
@ghwood ghwood added the bug Something isn't working label Apr 20, 2022
@jennifer-richards
Copy link
Member

I believe the 1970-01-01 date is because there is not yet an agenda for IETF 114. This should probably be indicated instead of displaying a bogus timestamp. For https://datatracker.ietf.org/meeting/113/agenda/, the date shows as "Updated: 2022-03-23 at 7:52:27 (PDT)" which is plausible (but still fixed to US Pacific time rather than something sensible for the user or meeting).

@jennifer-richards
Copy link
Member

Poking, there's an attempt to hide the "updated" indicator if meeting.updated() is not defined. Unfortunately, the updated() method returns 1970-01-01T00:00:00 rather thanNone in that situation, which defies the effort.

@rjsparks rjsparks added the contrib-medium Medium dev difficulty for contribution label Jul 20, 2022
@rjsparks
Copy link
Member

I think if you get back exactly that datetime, you an just say "it means this wasn't updated".
What will be returned when we become TZ aware?

@jennifer-richards
Copy link
Member

It localizes to PRODUCTION_TIMEZONE, so won't change due to the USE_TZ settings. It'd be nice to change it to be explicitly UTC instead of something that depends on settings.py if we want to use a magic number.

We could instead/also just say that anything before [date after Jan 2, 1970 and before plausible update time] is too long ago to be plausible. That'd avoid depending on a particular time zone for the view.

@Osoted56

This comment was marked as spam.

@microamp
Copy link
Contributor

Looks like there are some guards in place now.

if (!updatedDatetime.isValid || updatedDatetime < DateTime.fromISO('1980-01-01')) {
  return false
}

(in client/agenda/Agenda.vue)

It's hard to know when the change was made. The Git history only goes back to Feb 2024 if I'm reading it correctly.

I'll spend some time testing the code at the code sprint tomorrow.

@microamp microamp self-assigned this Jul 20, 2024
@microamp
Copy link
Contributor

It appears to me that when /api/meeting/114/agenda-data returns the default updated time, i.e. 1970-01-01T00:00:00Z, e.g.

{
    "meeting": {
        "number": "114",
        "city": "Philadelphia",
        "startDate": "2022-07-23",
        "endDate": "2022-07-29",
        "updated": "1970-01-01T00:00:00Z",
        "timezone": "America/New_York",
        "infoNote": "Please see the <a href=\"https://datatracker.ietf.org/meeting/114/proceedings\">IETF 114 Proceedings</a> for further records of this meeting.",
        "warningNote": ""
    },
    // ...
}

it correctly hides the Updated element in the UI.

image

So, I believe the bug has been fixed.

However, we may also want to apply the same filter to the plain text page for consistency.

^agenda(?P<utc>-utc)?(?P<ext>\.txt|\.csv)$

e.g. /meeting/114/agenda.txt

image

P.S. agenda.csv does not use updated, only agenda.txt does.

@rjsparks
Copy link
Member

rjsparks commented Jul 20, 2024

However, we may also want to apply the same filter to the plain text page for consistency.

Yes - that sounds good.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted bug Something isn't working component: meeting/ contrib-medium Medium dev difficulty for contribution minor
Projects
No open projects
Status: Looking for Sprinters
Development

Successfully merging a pull request may close this issue.

5 participants