Skip to content

Conversation

HonzaStefanik
Copy link
Contributor

@HonzaStefanik HonzaStefanik commented Jun 7, 2022

All issues should be addressed in appropriately named commits.

This one might need additional changes based on the requirements

@ViolanteCodes
Copy link
Collaborator

@HonzaStefanik quick question: in app.json there's a duplicate description field for the JAVA_BUTTER_CMS_API_KEY - i'm assuming it's okay for me to delete one of these?

@HonzaStefanik
Copy link
Contributor Author

Yes, I must've forgotten to delete one after rewriting the description

@ViolanteCodes ViolanteCodes self-requested a review June 20, 2022 15:32
@ViolanteCodes
Copy link
Collaborator

@HonzaStefanik Still not seeing the menus working: video. Other than that, the other issues look fixed, so I've closed them :)

Copy link
Collaborator

@ViolanteCodes ViolanteCodes left a comment

Choose a reason for hiding this comment

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

Menu still not working - see comment

@ViolanteCodes
Copy link
Collaborator

@HonzaStefanik Unfortunately, it looks like the "two-column-with-image" components are not rendering at all now: video

@ViolanteCodes ViolanteCodes self-requested a review July 5, 2022 15:31
@ViolanteCodes ViolanteCodes merged commit 8cfa86f into master Jul 5, 2022
@ViolanteCodes ViolanteCodes deleted the first-revision branch July 5, 2022 15:31
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