Skip to content

Conversation

@droidmonkey
Copy link
Member

Testing strategy

Added unit tests

Type of change

  • ✅ New feature (change that adds functionality)

@droidmonkey droidmonkey added this to the v2.7.10 milestone Nov 9, 2024
@droidmonkey droidmonkey force-pushed the feature/conv-regex-placeholder branch from 71b437c to 395c93f Compare November 10, 2024 14:10
@varjolintu varjolintu self-requested a review November 13, 2024 19:56
Copy link
Member

@varjolintu varjolintu left a comment

Choose a reason for hiding this comment

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

Works great. Tried it also with invalid values and ignored the UI warnings to check if the application complains about them when launching Auto-Type or opening a cmd URL.

* Closes #7293
* Move existing T-CONV and T-REPLACE-RX code from AutoType to Entry. Replumb AutoType to use the entry functions.
* Improve placeholder code in various place
@droidmonkey droidmonkey force-pushed the feature/conv-regex-placeholder branch from 395c93f to 1dd1039 Compare November 13, 2024 21:55
@codecov
Copy link

codecov bot commented Nov 13, 2024

Codecov Report

Attention: Patch coverage is 97.80220% with 2 lines in your changes missing coverage. Please review.

Project coverage is 63.87%. Comparing base (2fc24be) to head (1dd1039).
Report is 13 commits behind head on develop.

Files with missing lines Patch % Lines
src/core/Entry.cpp 97.59% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11453      +/-   ##
===========================================
+ Coverage    63.84%   63.87%   +0.03%     
===========================================
  Files          362      362              
  Lines        38115    38134      +19     
===========================================
+ Hits         24334    24358      +24     
+ Misses       13781    13776       -5     

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

@droidmonkey droidmonkey merged commit 9b8163c into develop Nov 13, 2024
4 checks passed
@droidmonkey droidmonkey deleted the feature/conv-regex-placeholder branch November 13, 2024 22:51
@phoerious phoerious added pr: new feature Pull request adds a new feature and removed new feature labels Nov 22, 2024
@droidmonkey droidmonkey added the pr: backported Pull request backported to previous release label Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: backported Pull request backported to previous release pr: new feature Pull request adds a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for {T-CONV:/Text/Type/} and {T-REPLACE-RX:/.../.../} as placeholder in fields

4 participants