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

Creating the form for adding a new course 🚀 #5

Merged
merged 11 commits into from
Oct 11, 2023

Conversation

ab-noori
Copy link
Owner

@ab-noori ab-noori commented Oct 9, 2023

EduPlanner Frontend: add a new course UI

🌟Features:

  • - The course form component for creating a new course is created and styled with bootstrap.
  • - The redux action and store for the form states are created using the redux toolkit.
  • - After successful submission of the form it will be redirected to the main page.
  • - The link to the course form component is added to the navigation panel.

🔗 Link to Backend PR

EduPlanner Backend: add a new course API

@gisachris gisachris self-requested a review October 11, 2023 09:21
Copy link
Collaborator

@gisachris gisachris left a comment

Choose a reason for hiding this comment

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

hello @partner,🤠

After careful review 🔍 and analysis 📜 , There is nothing left to implement 👨‍💻. You can merge This pull request :shipit: (Remarks below).

Requirements 🧰

  • Create the Form for Adding new Courses to the application. Creating the form for adding a new course 🚀  #5
  • Fetch data 📦 from the Rails API and manipulate it accordingly.
  • Follow design guidelines by this design on Behance.
  • Integrate Seemlessly(solve merge conflicts, etc) with User Authentication. 🔐.

Overview ☁️

  • @ab-noori Thank you for the great implementation of the Add Course page of the application. All requirements 🔼 above where met with great prescision and the team is lucky and proud to have you 🤝.

If you have any question feel free to ping @ me in this pull request or reach out on the other communication mediums we established in our working Agreement Thank you.

@ab-noori
Copy link
Owner Author

Hi @gisachris 👋,
Thank you so much 🙏 for reviewing and approving my pull request.
Your constructive and nice feedback is really appreciated 🙏.

thank-you-gif-22

@ab-noori ab-noori merged commit 29f7b7a into dev Oct 11, 2023
3 checks passed
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.

[4pt] Add class page
2 participants