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

'value' attribute in [experimental_filter_ability(_active)] can have 'default' value #8893

Merged

Conversation

newfrenchy83
Copy link
Contributor

for answer to an suggestion of @soliton- who think in filter abilitie, value must could check 'default' value of an abilitie/specials directly without have specify this value in filter

@github-actions github-actions bot added Units Issues that involve unit definitions or their implementation in the engine. Schema labels May 21, 2024
@newfrenchy83
Copy link
Contributor Author

@stevecotton could you review this PR, please? It remain 1 or 2 modification after this PR before what experimental_ can be removed.

src/utils/config_filters.cpp Outdated Show resolved Hide resolved
changelog_entries/filter_value_check_default.md Outdated Show resolved Hide resolved
data/schema/filters/abilities.cfg Outdated Show resolved Hide resolved
@newfrenchy83 newfrenchy83 force-pushed the add_default_value_in_filter_ability branch 3 times, most recently from 30b033c to 0dce3b7 Compare May 22, 2024 18:33
@soliton- soliton- requested a review from stevecotton May 23, 2024 12:09
@newfrenchy83 newfrenchy83 force-pushed the add_default_value_in_filter_ability branch from 0dce3b7 to 90f0289 Compare May 24, 2024 09:08
@newfrenchy83
Copy link
Contributor Author

@stevecotton could you merge this PR approved by @soliton- , please?

@stevecotton stevecotton merged commit 9fc5887 into wesnoth:master May 26, 2024
18 checks passed
@newfrenchy83 newfrenchy83 deleted the add_default_value_in_filter_ability branch May 26, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Schema Units Issues that involve unit definitions or their implementation in the engine.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants