Skip to content

Conversation

@godanny86
Copy link
Contributor

Description

Add finished versions of React and Angular projects to a single branch to promote ease of use.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

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

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Copy link

@habansal habansal left a comment

Choose a reason for hiding this comment

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

Nice consolidation under single branch :)

One question:
Is this migrated app commit a new one (new work in this PR) or picked from an existing branch: d0695c0

</profile>
<!-- AEM 6.x Profile to include Core Components-->
<profile>
<id>classic</id>

Choose a reason for hiding this comment

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

We don't need to a similar profile for React ?

Choose a reason for hiding this comment

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

@habansal react ...react/all/pom.xml does have the classic profile

https://github.com/adobe/aem-guides-wknd-spa/pull/21/files#diff-886c1a476526c03808b704eebb815fc298e130b3bf385ccd4f414c6f63ba485dR173-R179

Or are you referring to something else?

Copy link

@habansal habansal Jun 21, 2021

Choose a reason for hiding this comment

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

@davidjgonzalez I don't remember exactly what i noticed then.
But, looking at it again now, I see that there is indeed similar profile in React as well -

Thanks!

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.

3 participants