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

Other forms #77

Merged
merged 23 commits into from
Sep 11, 2024
Merged

Other forms #77

merged 23 commits into from
Sep 11, 2024

Conversation

Pramad712
Copy link
Contributor

Dance Club & Request Concert Functionality added!
(I tried to find a more gender-neutral icon for the dance club thing but the only other ones were a ball dance and pole dancing)
(there is also one small ui issue, that is the gap between each of the music pieces' text boxes, aka bottom margin, is 20, though in the code it is set to 0 (on all but the last one ofc to add a margin between it and the next question). 10 instead of 0 also doesn't work.)

@Pramad712 Pramad712 requested a review from TenType September 3, 2024 00:20
Copy link

github-actions bot commented Sep 3, 2024

🚀 Expo preview is ready!

  • Project → audacity-music-club
  • Platforms → android, ios
  • Runtime Version → 1.0.0
  • More info

Learn more about 𝝠 Expo Github Action

TenType
TenType previously requested changes Sep 4, 2024
Copy link
Member

@TenType TenType left a comment

Choose a reason for hiding this comment

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

Minor edits for readability and future debugging purposes (see discord)

Not fully tested yet

Pramad712 and others added 4 commits September 9, 2024 15:56
…ontent (performanceType dynamic changing still works)
- Fix circular dependency errors
- Fix LMH/MBT time limit validation
- Align with OOP by refactoring common methods into the base Form class
- Remove unused imports
- Organize imports
- Add required indicator to the RequestConcert time slot
Rename
- Rename Slot -> TimeSlotList to prevent ambiguity
@TenType TenType merged commit 7e37908 into main Sep 11, 2024
2 checks passed
@TenType TenType deleted the other-forms branch September 11, 2024 00:50
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