Skip to content

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jun 6, 2025

What is this pull request for?

Describe you pull request here...

Closes # (Remove if not related to any issue)

Notable changes (remove if none)

Explain any changes (maybe breaking?) that have been made.

Screenshots

Remove if no visual changes have been made.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen added this to the 8.0 milestone Jun 6, 2025
@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.11%. Comparing base (151dd07) to head (d198a96).

Files with missing lines Patch % Lines
...llers/alchemy/admin/page_definitions_controller.rb 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3262      +/-   ##
==========================================
- Coverage   97.14%   97.11%   -0.03%     
==========================================
  Files         279      280       +1     
  Lines        7378     7388      +10     
==========================================
+ Hits         7167     7175       +8     
- Misses        211      213       +2     

☔ 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.

@tvdeyen tvdeyen force-pushed the definitions-module branch from 39bb57b to 317d6b7 Compare June 10, 2025 13:24
@tvdeyen tvdeyen force-pushed the definitions-module branch 3 times, most recently from 6becdc7 to 502f3ab Compare June 11, 2025 15:12
@tvdeyen tvdeyen force-pushed the definitions-module branch 3 times, most recently from acd859c to 2ed70b9 Compare June 11, 2025 20:44
@tvdeyen tvdeyen force-pushed the definitions-module branch from 2ed70b9 to 1f800f4 Compare June 12, 2025 13:49
@github-actions
Copy link

This pull request has not seen any activiy in a long time.
Probably because of missing tests or a necessary rebase.
This PR will be closed in 7 days if no further activity happens.

@github-actions github-actions bot added the Stale label Aug 12, 2025
@tvdeyen tvdeyen force-pushed the definitions-module branch from 1f800f4 to 78e9e2b Compare August 12, 2025 13:34
@github-actions github-actions bot removed the Stale label Aug 13, 2025
@tvdeyen tvdeyen removed this from the 8.0 milestone Sep 2, 2025
@tvdeyen tvdeyen mentioned this pull request Sep 2, 2025
3 tasks
@tvdeyen tvdeyen force-pushed the definitions-module branch 2 times, most recently from 3834854 to 7383854 Compare September 7, 2025 20:34
@tvdeyen tvdeyen force-pushed the definitions-module branch 3 times, most recently from e1e954d to 955a712 Compare September 18, 2025 14:09
@tvdeyen tvdeyen force-pushed the definitions-module branch 2 times, most recently from dbdf707 to 17f8267 Compare October 4, 2025 15:46
@tvdeyen tvdeyen force-pushed the definitions-module branch 3 times, most recently from 3d089ac to e1c7ed4 Compare October 26, 2025 16:22
@tvdeyen tvdeyen force-pushed the definitions-module branch from e1c7ed4 to d198a96 Compare October 26, 2025 18:05
<li>
<header>
<strong><%= Alchemy::Ingredient.translated_label_for(ingredient_definition.role, element_name) %></strong>
<small><%= Alchemy::Ingredient.normalize_type(ingredient_definition.type).constantize.model_name.human %></small>

Check warning

Code scanning / Brakeman

Unsafe reflection method constantize called on model attribute. Warning

Unsafe reflection method constantize called on model attribute.
Makes it much easier to choose the right element
Helps to visualize the usage of the page.
This is the know icon and only if we have a custom icon we
should change it.
Determines the icon name for an element based on its name.
Can be configured via +Alchemy.config.element_icon_finder+.
The default implementation uses simple pattern matching on remixicon names.
@tvdeyen tvdeyen force-pushed the definitions-module branch from d198a96 to e148eb1 Compare October 28, 2025 23:25
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.

2 participants