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

#536289: [SXA] Add custom template for _jss scaffold_ #1420

Merged

Conversation

sc-ruslanmatkovskyi
Copy link
Collaborator

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi commented Apr 5, 2023

Description / Motivation

Added nextjs-sxa scaffold component action in initializer.

Testing Details

  • Unit Test Added
  • Manual Test/Other (Please elaborate)

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)

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from ea46aae to 812fa72 Compare April 5, 2023 16:33
Copy link
Contributor

@ambrauer ambrauer left a comment

Choose a reason for hiding this comment

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

@matkovskyi any reason not to just include this with the existing nextjs-sxa initializer add-on?

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from 812fa72 to a8b8282 Compare April 6, 2023 06:55
@sc-ruslanmatkovskyi
Copy link
Collaborator Author

@matkovskyi any reason not to just include this with the existing nextjs-sxa initializer add-on?

Yes, you're right! I fixed it

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from a8b8282 to aa075a0 Compare April 6, 2023 07:00
@sc-ruslanmatkovskyi sc-ruslanmatkovskyi changed the title #536289: [SXA] added nextjs-sxa-scaffold initializer #536289: [SXA] added nextjs-sxa scaffold in initializer Apr 6, 2023
@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from aa075a0 to c096f2e Compare April 6, 2023 10:49
@illiakovalenko
Copy link
Contributor

illiakovalenko commented Apr 6, 2023

@matkovskyi Please, also update CHANGELOG.md file to include your PR. We started to update Changelog for each PR, so it makes easier for us to start release processes
And fill required checkboxes, provide description in the PR template

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from cf0bdc8 to 099100a Compare April 6, 2023 11:32
@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the feature/536289-implement-jss-scaffold-sxa-initializer branch from 099100a to cbd2829 Compare April 6, 2023 11:34
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@illiakovalenko illiakovalenko changed the title #536289: [SXA] added nextjs-sxa scaffold in initializer #536289: [SXA] Add custom template for _jss scaffold_ Apr 6, 2023
@illiakovalenko illiakovalenko merged commit 0eca08c into dev Apr 6, 2023
@illiakovalenko illiakovalenko deleted the feature/536289-implement-jss-scaffold-sxa-initializer branch April 6, 2023 11:58
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.

4 participants