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

FEAT: Activity Creation #731

Merged
merged 111 commits into from
Sep 7, 2023

Conversation

yaya-mamoudou
Copy link
Contributor

@yaya-mamoudou yaya-mamoudou commented Sep 3, 2023

Summary

This PR has the new screens for activity creation and activity details.

Closes #557

Changes

  • New form for Activity Creation
  • Added preview for on the activity form after every step submission
  • New page for Activity details
  • Made changes to the backend to accommodate new fields.
  • modified result logic when user updates an activity

Screenshots

image
image
image
image
image

… multiple prop is set and also add checkboxes when the withCheckbox property is set
…dstudio#714)

* Fixed issue with comment form submitting when empty

* Add validation of username when creating new creator to make sure username is not entered as an email

* Fixed issue with comment form submitting when empty

* -

* Wrap error string with translation method
@tuxology
Copy link
Member

tuxology commented Sep 4, 2023

@yaya-mamoudou Thanks for this PR! Most stuff works well 🎉 I am listing some TODOs here that still need some fixing/adjustment

Blockers

  • "Download PDF" button is not working

Bugs/Improvements

  • Clicking on photos not bring up modal
  • Increase min height of images in each step/introduction - it's too thin right now - if a single or two images are added, they look cropped
  • Categories, Class Grade "tags" are buttons - it seems they are clickable. We should change that to be not clickable and non hover able. This is observed in Project tags and categories as well
  • /activities URL takes you to activities list but not enough padding there. We could add a empty div with padding for this section and then place the activities component there?

Copy link
Member

@tuxology tuxology left a comment

Choose a reason for hiding this comment

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

Even though general flow is ok and blockers are removed, there is a lot of dead-code and comments left. We need to remove it later. Approving and merging this for now.

@tuxology tuxology merged commit baf7051 into unstructuredstudio:master Sep 7, 2023
iamhks added a commit to iamhks/zubhub that referenced this pull request Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api backend issue client Frontend issue feature Outreachy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve the UI/UX for activity and project creation features for ZubHub
2 participants