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

DDFFORM 417 display opening hours #937

Merged
merged 6 commits into from
Apr 5, 2024

Conversation

LasseStaus
Copy link
Contributor

@LasseStaus LasseStaus commented Apr 3, 2024

Link to issue

Jira: DDFFORM-417

This PR depends on two PRs.

Design system PR
React PR

Description

This PR adds a paragraph for displaying the openingHours for a branch.

It should only be possible to use this paragraph on the content type 'Branch'

Most of the logic is done in the react app, commits should explain what is being done here.

Screenshot of the result

image

Additional comments or questions

Consider if it is necessary to add more checks in the preprocess.
Consider if the branch_id is retrieved properly.

This ensures that the paragraph cannot be selected for the content types Artciel and Page, as it should only be used on the content-type Branch

DDFFORM-417
@github-actions github-actions bot temporarily deployed to pr-937 April 3, 2024 18:45 Destroyed
@LasseStaus LasseStaus changed the base branch from develop to opening-hours-api April 3, 2024 18:45
@LasseStaus LasseStaus force-pushed the DDFFORM-417-display-opening-hours branch from a8c038d to cf10c6e Compare April 3, 2024 18:55
@github-actions github-actions bot temporarily deployed to pr-937 April 3, 2024 18:55 Destroyed
Copy link
Contributor

@kasperg kasperg left a comment

Choose a reason for hiding this comment

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

👍 I have one consideration for you. In general I think this looks good!

web/modules/custom/dpl_paragraphs/dpl_paragraphs.module Outdated Show resolved Hide resolved
web/modules/custom/dpl_paragraphs/dpl_paragraphs.module Outdated Show resolved Hide resolved
@kasperg kasperg assigned LasseStaus and unassigned kasperg, rasben and kasperbirch1 Apr 4, 2024
@LasseStaus LasseStaus force-pushed the DDFFORM-417-display-opening-hours branch from cf10c6e to 2693fa7 Compare April 4, 2024 16:20
@github-actions github-actions bot temporarily deployed to pr-937 April 4, 2024 16:20 Destroyed
This is added to a new dpl_opening_hours.module file to keep paragraph context related to opening hours.

This prepares the data necessary for the react app opening hours.

DDFFORM-417
This is all generated from task dev:codegen:dpl-cms

DDFFORM-417
@LasseStaus LasseStaus force-pushed the DDFFORM-417-display-opening-hours branch from 2693fa7 to 29c79fc Compare April 5, 2024 11:31
@github-actions github-actions bot temporarily deployed to pr-937 April 5, 2024 11:31 Destroyed
@LasseStaus LasseStaus merged commit cb85988 into opening-hours-api Apr 5, 2024
16 checks passed
@LasseStaus LasseStaus deleted the DDFFORM-417-display-opening-hours branch April 5, 2024 11:45
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.

4 participants