Angeline assignment2 #2
Conversation
There was a problem hiding this comment.
Make sure you don't commit .env files! Add .env to you .gitignore file.
| const fetchPostData = async () => { | ||
| try { | ||
| const data = await getPostData(); | ||
| setPostData(data); | ||
| } catch (error) { | ||
| console.error('Error fetching data:', error); | ||
| } | ||
| }; |
There was a problem hiding this comment.
I don't know if you necessarily need a fetchPostData and getPostData function in the same file. Usually you'd store the getPostData function in a /supabase/post.ts queries file.
| <View style={styles.commentSection}> | ||
| {/* First Comment */} | ||
| <View style={styles.comment}> | ||
| <ProfilePlaceholder style={styles.commentIcon} /> | ||
| <View style={styles.commentBody}> | ||
| <View style={styles.commentHeader}> | ||
| <Text style={styles.commentName}>philip_ye</Text> | ||
| <Text style={styles.commentDate}>September 20</Text> | ||
| </View> | ||
| <Text style={styles.commentText}> | ||
| This organization is doing amazing work tackling the complex | ||
| root causes of the issue. | ||
| </Text> | ||
| </View> | ||
| </View> |
There was a problem hiding this comment.
Given that there are so many divs for each div, might be helpful to create a separate Comment component.
There's also a comments table that you can pull this data from!
| borderTopColor: '#ccc', | ||
| paddingTop: 10, | ||
| marginTop: 20, | ||
| paddingRight: 10, |
There was a problem hiding this comment.
Avoid asymmetric spacing attributes. See if you can achieve this styling using flexbox properties.
| }, | ||
| postDate: { | ||
| color: 'gray', | ||
| paddingLeft: 120, |
There was a problem hiding this comment.
You should be able to create an offset like this using justifyContent and alignItems attributes.
What's new in this PR
Description
pulled post data from supabase; fixed styling
Screenshots
How to review
Next steps
Relevant links
Online sources
Related PRs
CC: @ronniebeggs