-
Notifications
You must be signed in to change notification settings - Fork 6
Address issues from the first revision of the app #9
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
Conversation
…d of one per line
…va command, minor cosmetic changes
…it as arguments instead
@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? |
Yes, I must've forgotten to delete one after rewriting the description |
@HonzaStefanik Still not seeing the menus working: video. Other than that, the other issues look fixed, so I've closed them :) |
There was a problem hiding this 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
…lay query parameters in search result URL
@HonzaStefanik Unfortunately, it looks like the "two-column-with-image" components are not rendering at all now: video |
All issues should be addressed in appropriately named commits.
This one might need additional changes based on the requirements