Skip to content

Conversation

@muneebGH
Copy link
Contributor

@muneebGH muneebGH commented Jul 4, 2022

Description

Description of changes made

Ticket Link

Link to the associated ticket
Link title

Post-review

  • Squash commits into discrete sets of changes
  • Ensure that once the changes have been deployed to stage, prod is manually deployed

Returns:
bool: True if available via association else False
"""
return bool(pathway.get('visible_via_association', False))
Copy link
Member

Choose a reason for hiding this comment

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

visible_via_association is already a boolean field, Why do we need this method?

'card_image_url': get_pathway_card_image_url(searchable_product),
'partners': get_pathway_partners(searchable_product),
'subjects': get_pathway_subjects(searchable_product),
'visible_via_association': get_pathway_association(searchable_product),
Copy link
Member

Choose a reason for hiding this comment

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

visible_via_association should be already available here in searchable_product as a boolean field.

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.

3 participants