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

CodyGarciaa/Prea-Pages-Cont #20

Merged
merged 40 commits into from
Nov 1, 2024
Merged

CodyGarciaa/Prea-Pages-Cont #20

merged 40 commits into from
Nov 1, 2024

Conversation

CodyGarciaa
Copy link
Contributor

What's new in this PR

Description

  • the pages that show up on the PreaPages screen are now reflective of what is in our supabase

  • the posters are still the same, but the names and data connected to each page is based on the supabase

  • also, the spanish and english tabs work now, only having every page under its relative language according to the supabase

  • also, the pages are put in order before rendered on the screen

  • for the video page side, the video plays in relation to which page is pressed

  • the next and prev buttons now work for the next page in the order of pages in the relative language

Screenshots

IMG_0653
IMG_0652

How to review

  • review LegalRights's index.tsx, and then VideoPage's index.tsx
  • had to change things up in LegalRights first, and then work to pass data to VideoPage from LegalRights onPress
  • so sorry there's no commits because I was accidentally coding/committing on my remote main rip
  • feed back on: does all the videos play all the time? when I was testing, it's like 60% of the time playing on my end

Next steps

Relevant links

Online sources

Related PRs

  • erm maybe conflict with Charlotte's navigation, lowk hella hardcoded my navigation in app.tsx bc u said so :)

CC: @philipye314

angelinetu and others added 5 commits October 26, 2024 15:29
* seek help to resource list

* seek help to resource list

* fixed some things

* removed unused import

---------

Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>
* [feat] resources rendered and scrolling by buttons

* good work arfa, minor adjustments

---------

Co-authored-by: philipye314 <[email protected]>
* navigation

* got rid of unused files

* CodyGarciaa/Legal-Rights-Prea-Pages (#13)

* set up legal rights skeleton, can go to video page with nothing there yet tho

* added video and testing player

* more testing with video, lowk not getting .mp4 to work

* legal rights page scrollable

* added caption buttons, lowk still to add more to it

* closed caption buttons are toggleable and XOR

* finished pr

* resolved philip's problems lol

* 5 create resources sub pages (#15)

* seek help to resource list

* seek help to resource list

* fixed some things

* removed unused import

---------

Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>

* 8-healing-resource-pages (#19)

* [feat] resources rendered and scrolling by buttons

* good work arfa, minor adjustments

---------

Co-authored-by: philipye314 <[email protected]>

* [bug fix] removed extra type file

* navigation

* got rid of unused files

* [fix] 🐐🐐🐐

* [fix] did screen props

* deleted unused files

---------

Co-authored-by: philipye314 <[email protected]>
Co-authored-by: CodyGarciaa <[email protected]>
Co-authored-by: angelinetu <[email protected]>
Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>
Co-authored-by: Arfa Momin <[email protected]>
Copy link
Collaborator

@philipye314 philipye314 left a comment

Choose a reason for hiding this comment

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

looks good, just some minor changes to fix

CharlotteLaw and others added 3 commits October 26, 2024 16:17
* navigation

* got rid of unused files

* CodyGarciaa/Legal-Rights-Prea-Pages (#13)

* set up legal rights skeleton, can go to video page with nothing there yet tho

* added video and testing player

* more testing with video, lowk not getting .mp4 to work

* legal rights page scrollable

* added caption buttons, lowk still to add more to it

* closed caption buttons are toggleable and XOR

* finished pr

* resolved philip's problems lol

* 5 create resources sub pages (#15)

* seek help to resource list

* seek help to resource list

* fixed some things

* removed unused import

---------

Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>

* 8-healing-resource-pages (#19)

* [feat] resources rendered and scrolling by buttons

* good work arfa, minor adjustments

---------

Co-authored-by: philipye314 <[email protected]>

* [bug fix] removed extra type file

* navigation

* got rid of unused files

* [fix] 🐐🐐🐐

* [fix] did screen props

* deleted unused files

---------

Co-authored-by: philipye314 <[email protected]>
Co-authored-by: CodyGarciaa <[email protected]>
Co-authored-by: angelinetu <[email protected]>
Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>
Co-authored-by: Arfa Momin <[email protected]>
Copy link
Collaborator

@philipye314 philipye314 left a comment

Choose a reason for hiding this comment

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

I started doing work on some stuff (mainly pulling main into your branch), so be sure to try and match all the different typing now that we have nav. Let me/Charlotte know if you have any questions?

CodyGarciaa and others added 16 commits October 31, 2024 18:52
…not playing anymore. but all the interactivity with supabase and within the app works lol
* navigation

* got rid of unused files

* CodyGarciaa/Legal-Rights-Prea-Pages (#13)

* set up legal rights skeleton, can go to video page with nothing there yet tho

* added video and testing player

* more testing with video, lowk not getting .mp4 to work

* legal rights page scrollable

* added caption buttons, lowk still to add more to it

* closed caption buttons are toggleable and XOR

* finished pr

* resolved philip's problems lol

* 5 create resources sub pages (#15)

* seek help to resource list

* seek help to resource list

* fixed some things

* removed unused import

---------

Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>

* 8-healing-resource-pages (#19)

* [feat] resources rendered and scrolling by buttons

* good work arfa, minor adjustments

---------

Co-authored-by: philipye314 <[email protected]>

* [bug fix] removed extra type file

* navigation

* got rid of unused files

* [fix] 🐐🐐🐐

* [fix] did screen props

* deleted unused files

---------

Co-authored-by: philipye314 <[email protected]>
Co-authored-by: CodyGarciaa <[email protected]>
Co-authored-by: angelinetu <[email protected]>
Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>
Co-authored-by: Arfa Momin <[email protected]>
* navigation

* got rid of unused files

* CodyGarciaa/Legal-Rights-Prea-Pages (#13)

* set up legal rights skeleton, can go to video page with nothing there yet tho

* added video and testing player

* more testing with video, lowk not getting .mp4 to work

* legal rights page scrollable

* added caption buttons, lowk still to add more to it

* closed caption buttons are toggleable and XOR

* finished pr

* resolved philip's problems lol

* 5 create resources sub pages (#15)

* seek help to resource list

* seek help to resource list

* fixed some things

* removed unused import

---------

Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>

* 8-healing-resource-pages (#19)

* [feat] resources rendered and scrolling by buttons

* good work arfa, minor adjustments

---------

Co-authored-by: philipye314 <[email protected]>

* [bug fix] removed extra type file

* navigation

* got rid of unused files

* [fix] 🐐🐐🐐

* [fix] did screen props

* deleted unused files

---------

Co-authored-by: philipye314 <[email protected]>
Co-authored-by: CodyGarciaa <[email protected]>
Co-authored-by: angelinetu <[email protected]>
Co-authored-by: Philip Ye <[email protected]>
Co-authored-by: philipye314 <[email protected]>
Co-authored-by: Arfa Momin <[email protected]>
philipye314
philipye314 previously approved these changes Nov 1, 2024
Copy link
Collaborator

@philipye314 philipye314 left a comment

Choose a reason for hiding this comment

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

looks good. We'll talk a little bit about styling + supabase changes in the next sprint but this is very good work overall!

@CodyGarciaa CodyGarciaa dismissed philipye314’s stale review November 1, 2024 02:01

The merge-base changed after approval.

@philipye314 philipye314 merged commit 98a8000 into main Nov 1, 2024
2 checks passed
@philipye314 philipye314 deleted the 14-prea-videos-contd branch November 1, 2024 02:01
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.

5 participants