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

Remove redundant code (handlebars values) #128

Merged
merged 7 commits into from
Oct 21, 2021
Merged

Conversation

unseen1980
Copy link
Member

This PR removes redundant handlebars values that are not required anymore. This will help us fix the integration test in Fiori Freestyle generator.

@changeset-bot
Copy link

changeset-bot bot commented Oct 20, 2021

⚠️ No Changeset found

Latest commit: dfca34f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

IainSAP
IainSAP previously approved these changes Oct 20, 2021
tobiasqueck
tobiasqueck previously approved these changes Oct 21, 2021
Copy link
Contributor

@tobiasqueck tobiasqueck left a comment

Choose a reason for hiding this comment

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

Text improvements and removal of obsolete handlebar code looks good.

@unseen1980 unseen1980 merged commit 3ac4650 into main Oct 21, 2021
@unseen1980 unseen1980 deleted the fix/redundant-code branch October 21, 2021 12:58
@unseen1980
Copy link
Member Author

Thanks!

devinea added a commit that referenced this pull request Oct 23, 2021
* origin/main:
  chore: apply latest changesets
  Fix versions (#146)
  chore: apply latest changesets
  Fix missing client in fiori proxy config (#138)
  fix(odata-service-writer): Reinstate yaml config ordering (#140)
  chore: apply latest changesets
  chore: update version pipeline to skip git hooks (#141)
  chore: update version pipeline to use github PAT (#137)
  feat: enable changesets and release workflow (#129)
  fix(odata-service): Dont add mock config if no metadata (#135)
  refactor: remove common module (#122)
  Remove redundant code (handlebars values) (#128)
  fix(odata-service): Removes ui5-config -> odata-service-writer dep (#127)
  fix(ui5-application-writer): Fix error when running lint task of app (#125)
  refactor: rm common types module (#123)

# Conflicts:
#	.github/workflows/pipeline.yml
#	package.json
#	packages/odata-service-writer/src/index.ts
#	pnpm-lock.yaml
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.

5 participants