Skip to content

Feat/#27 create home page#112

Open
AlexandruIstrate wants to merge 17 commits intodevelopmentfrom
feat/#27-create-home-page
Open

Feat/#27 create home page#112
AlexandruIstrate wants to merge 17 commits intodevelopmentfrom
feat/#27-create-home-page

Conversation

@AlexandruIstrate
Copy link

@AlexandruIstrate AlexandruIstrate commented Feb 20, 2022

This pull request solves issue:

Closes: #27

Which app is this issue related to?

  • REST API
  • E-Learning React App
  • Admin Panel - Not created yet

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Wiki Updates

  • No need for updates
  • Requires an update ( not done within this issue )
  • Has been updated ( done within this issue )

Testing checklist

  • Unit tests
  • Functional tests
  • E2E tests

Security Filters - For API Only

Please make sure to add the respective Security Filters before submitting a PR

  • They have been added within this issue.

Compatibility tests - For E-Learning React App Only

  • Tested on Safari
  • Tested on Chrome
  • Tested on Mozilla Firefox

What changes have been done within this issue?

This issue creates the home page for the REACT app.

How should this be manually tested?

Open the page in the browser and see if everything looks right?

Screenshots or example usage

@AlexandruIstrate AlexandruIstrate added the feat 🎸 A new feature label Feb 20, 2022
@AlexandruIstrate AlexandruIstrate self-assigned this Feb 20, 2022
@rmrn65 rmrn65 added the e-learning-app 🎒 Changes on the E-Learning-App label Feb 20, 2022
Copy link
Contributor

@CristiPV CristiPV left a comment

Choose a reason for hiding this comment

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

There are unused imports on the home page and the card carousel components which should be removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e-learning-app 🎒 Changes on the E-Learning-App feat 🎸 A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Home page

3 participants