Community group page detail#654
Conversation
Need updated API settings.
0xZakk
left a comment
0xZakk
left a comment
There was a problem hiding this comment.
@secondaryfun Really impressive work here! This looks really good! See my feedback below
…tform/CoTrip into community-group-page-detail
0xZakk
left a comment
0xZakk
left a comment
There was a problem hiding this comment.
Getting the following error when I try to go to the explore-people page:
Getting a lot of other errors too. Can you go through and double check everything is working before requesting another revew?
|
@secondaryfun waiting on a PR review from @Underwaterr after that you should be able to pull from develop to update that page or you could pull directly from community-group-detail-search which is updated as well |
|
Hopefully its working now. I identified an issue with the nav-bar, which fails to load a placeholder image, but i'll address that in a separate issue. |
0xZakk
left a comment
0xZakk
left a comment
There was a problem hiding this comment.
@secondaryfun Can you test to make sure everything is working before requesting a review?
|
Changes to the Member-Page are simply to make the placeholder data work with the correct settings on my component app. |
0xZakk
left a comment
0xZakk
left a comment
There was a problem hiding this comment.
before we can merge, fix issue with loading group detail page
3 participants
Related Issue (include '#'): Creating Detailed GROUP page for Community #580
Description of changes made:
** Title
** Post
** Comments
** Description
** Members
Features remaining to add as issues:
** View upcoming events for the group.
** View group media.
** Replies to comments
** Add comments via button
** Add likes to posts or comments
Is there specific feedback you would like on these changes:
Screenshot(s):
