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

style: ergonomic design form #16

Merged
merged 18 commits into from
Jul 25, 2023
Merged

style: ergonomic design form #16

merged 18 commits into from
Jul 25, 2023

Conversation

julianramirez2
Copy link
Contributor

@julianramirez2 julianramirez2 commented Jul 21, 2023

Description

The form's ergonomics were enhanced to make it more user-friendly. The action selection area and the button to execute the actions were integrated into the same form to avoid disconnection. Additionally, a box was added to hold logs in the future (there is a temporary function that generates logs). Furthermore, the message that previously said "Post updated" was modified to "Enrollment action executed."

Testing instructions

To test, simply access the form, fill in or modify the information, and create/update the enrollment.

Additional information

The get_logs function will be temporarily present only as an example to showcase the visualization. Once the logging functionality is developed, these placeholder logs will be replaced with the real ones.

Checklist for Merge

  • Tested in a remote environment
  • Updated documentation
  • Rebased master/main
  • Squashed commits

@julianramirez2 julianramirez2 requested a review from MaferMazu July 22, 2023 02:10
Copy link
Contributor

@MaferMazu MaferMazu 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 to me @julianramirez2, please remember squash and merge.

@julianramirez2 julianramirez2 merged commit e5215ab into main Jul 25, 2023
@felipemontoya felipemontoya deleted the ergonomic-design-form branch March 21, 2024 17:31
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