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

FOUR-14237 Fix Iris Bank Problem with TAG a href #421

Merged
merged 1 commit into from
Jun 10, 2024
Merged

Conversation

caleeli
Copy link
Contributor

@caleeli caleeli commented May 20, 2024

Fix Iris Bank Problem with TAG a href

Steps to Reproduce:

  1. log in ProcessMaker, go to Designer click on +screen
  2. create a new screen type EMAIL
  3. use a Rich text with LINK
  4. now click on the element you see a menu click on HREF

Current Behavior:

You will see that the domain URL is automatically filled in, and if you click on CANCEL you will see that the URL is applied to the element href

Expected Behavior:

It should detect that the href is not empty and should not add anything.

Related tickets

ci:next
ci:deploy

@caleeli
Copy link
Contributor Author

caleeli commented May 20, 2024

FixRichTextLinkEditor.mp4

@tokensPM
Copy link

QA server K8S was successfully deployed https://ci-f8daab0b8d.engk8s.processmaker.net

Copy link

@devmiguelangel devmiguelangel left a comment

Choose a reason for hiding this comment

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

Screenshot from 2024-05-21 08-09-44

@tokensPM
Copy link

QA server K8S was successfully deployed https://ci-f8daab0b8d.engk8s.processmaker.net

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
0.0% 0.0% Duplication on New Code

See analysis details on SonarQube

@ryancooley ryancooley merged commit 12fd5dd into next Jun 10, 2024
11 of 15 checks passed
@ryancooley ryancooley deleted the bugfix/FOUR-14237 branch June 10, 2024 02:09
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.

4 participants