search

Metro-Records / la-metro-councilmatic

:metro: An instance of councilmatic for LA Metro
MIT License
6 stars 2 forks source link

Unpopulated "Significant Date" facet #716

Closed fgomez828 closed 2 years ago

fgomez828 commented 3 years ago

Hi @shrayshray! In addressing the facet ordering issue, I noticed the "Significant Date" facet isn't populating. Do you have any info about whether you expect this facet to be populated or not?

shrayshray commented 3 years ago

Hi, @fatima3558! "Significant Date" is the label we want to use for the facet, but the Class is actually "Date". I think we changed the class name at some point because it caused confusion on this issue, too.

hancush commented 3 years ago

Looks like we have Dates plural!

https://github.com/datamade/la-metro-councilmatic/blob/8a83113363e586e6853532a326dbda0239f5a892/lametro/management/commands/refresh_guid.py#L70-L72

Hopefully changing to singular will resolve issue.

hancush commented 2 years ago

@shrayshray This is also done and deployed to staging. Care to double check the significant date facet?

shrayshray commented 2 years ago

@hancush fantastic, thank you!

hancush commented 2 years ago

Also published, with new values to appear at next reindex. 😎