Skip to content

feat(item.tpl): rendering 'dir' if set as attribute#2367

Open
albertosgz wants to merge 1 commit intodevelopfrom
feature/AUT-2939
Open

feat(item.tpl): rendering 'dir' if set as attribute#2367
albertosgz wants to merge 1 commit intodevelopfrom
feature/AUT-2939

Conversation

@albertosgz
Copy link
Copy Markdown
Contributor

@albertosgz albertosgz commented May 26, 2023

Fixes AUT-2939

What's changed

Item template renders dir when is setup properly.

Needed for oat-sa/extension-tao-mediamanager#470

@github-actions
Copy link
Copy Markdown

Version

Target Version 30.2.0
Last version 30.1.0

There are 0 BREAKING CHANGE, 1 feature, 0 fix

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 26, 2023

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.37%. Comparing base (815b53a) to head (5ce1505).
⚠️ Report is 2090 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #2367   +/-   ##
==========================================
  Coverage      16.37%   16.37%           
  Complexity      4207     4207           
==========================================
  Files            389      389           
  Lines          12838    12838           
==========================================
  Hits            2102     2102           
  Misses         10736    10736           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@albertosgz albertosgz requested a review from pnal May 29, 2023 13:21
Copy link
Copy Markdown
Contributor

@pnal pnal left a comment

Choose a reason for hiding this comment

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

  • New code is covered by tests (if applicable)
  • Tests are running successfully (old and new ones) on my local machine (if applicable)
  • New code is respecting code style rules
  • New code is respecting best practices
  • New code is not subject to concurrency issues (if applicable)
  • Feature is working correctly on my local machine (if applicable)
  • Acceptance criteria are respected
  • Pull request title and description are meaningful
  • Pull request's target is not master

@swiec
Copy link
Copy Markdown

swiec commented Oct 3, 2023

@mike-ionut-mihai-sandu-tao just to remind you, that you still didn't pay my invoice.

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