fix(FSADT1-848): fixing external form style #568
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
javascript
Current changelogBug Fixes
Features
Reverts |
github-actions
bot
added
github_actions
github-actions
bot
added
java
Import missing variable.
|
[nr-forest-client-backend] Kudos, SonarCloud Quality Gate passed! |
|
[forest-client-frontend] SonarCloud Quality Gate failed.
|
|
[nr-forest-client-processor] Kudos, SonarCloud Quality Gate passed!
|
1 task
paulushcgcj
added a commit
that referenced
this pull request
Oct 23, 2023
* fix: fixing progress state when error * fix: fixing error notification Fixing error notification below progress indication. * fix: fixing dropdown * fix: fixing multi select * chore: updating stubs to add more addresses * fix: fixing dropdown not changing when selection change * feat: add formErrors variable * chore: redoing css changes - Part #1 * feat: display address extra errors * feat: move extra errors to the error component * chore: redoing css changes - Part #2 * feat: replace formErrors argument with notification approach * feat: scroll to new contact * feat: add callback argument to useFocus * feat: make the bindings to create and scroll to new contact * chore: minor adjusments on useFocus * chore: redoing css changes - Part #3 * chore: redoing css changes - Part #4 * test: test the validation function and the error component * fix: update contactType value on select * test: ignore already valid location * test: remove outdated test * feat: add parameter for the shadow directive * feat: display progress indicator vertically Also adds some changes for displaying it horizontally. * FSADT1-868: Allowed special characters. Done olnly in the Frontend as the Backend was already allowing special charactes. * test: add getMany command * test: add progress indicator tests * fixed inline notification colors * Removed wrong style * Added property needed * feat: show logout modal * refactor: unify code for the logout button in different sizes * feat: show help modal * feat: use media query instead of outerWidth to determine screen size * Removed class that was causing issues with the logout button * Revert "Removed class that was causing issues with the logout button" This reverts commit 827e496. * Fixed logout button * feat: adjust header buttons * Fixed mask issue with the postal code * Renamed some files before refactoring code * no message * Made code reviews * chore: remove console.log * fix: prevent an older request from overwriting the results * Made requiered changes * Removed repeatead code * Removed unneeded setting * fix: do not always block response from older request It should be allowed when no newer request was yet responded. * chore: use simple counter to generate requestId * fix: init fetching * test: fix test * Fixed file * Added javadoc comment as per requested by SonarCloud * docs: improve comment * feat: adjust header logo and buttons * feat: adjust header spaces and breakpoints * chore: use date milliseconds as id * fix: fix build issue * fix: prevent error in the workflow * fix: attempt to fix tests in the workflow * fix: remove package from optimizeDeps As an attempt to prevent error in the workflow. * refactor: requested changes from review * Refactored code - Part #1 * Made code reviews * Formatted and beautified code * style: small changes to code style * chore: fix code linter * Refactored code - Part #1.1 * fix: Fixed sonarcloud security hotspot issue * fix: Fixed security hotspot * Refactored code - Part #2 * Refactored code - Part #2.1 * fix: fix header small screen * fix: attempt to fix side effect in the workflow * chore: submitting small changes * fix(FSADT1-884): fixing steps being enable - adjusted the steps to be disable by default - only enable if valid and only for the next step * chore: sorting import * fix(FSADT1-887): fixing typo on step 4 * fix(FSADT1-889): fixing font style * fix: Fixed security hotspot and improved wording * fix: Fixed security hotspot * fix(FSADT1-888): fixing spacing between radio buttons * Trying a different approach to solve the security hotspot issue * fix(FSADT1-892): fixing submission page * Trying a different approach to solve the security hotspot issue #2 * fix(FSADT1-894): fixing field spacing and width * fix(FSADT1-891): fixing modal spacing * fix(FSADT1-889): fixing notification body content * fix(FSADT1-890): fixing component alignment * fix: Fixed security hotspot * fix: Fixed security hotspot * Made code reviews * Removed unused variable * Refactored code and made code reviews * feat: adjust margins * fix(FSADT1-884): removing navigation link * Made code reviews * fix: header height * chore: replace mock e-mail from .local to .com * fix: do not pre-validate current step * feat: replace margin with padding * FSADT1-899: Removed unneeded validation * ci: updating trivy to fix issues * no message * no message * Tried to skip SonarCloud validation * Skipped SonarCloud validation * chore: adding us address stub data * fix(FSADT1-884): fixing subtitle style * fix: autocomplete after changing country * fix: fix subtitle style for vertical progress indicator * fis(FSADT1-894): fixing style for inline and text * Made code reviews * FSADT1-879 * - Made code reviews - Removed unused code * Made code reviews * Made code reviews * Made improvement * feat: clear text displayed on combo-box * fix: clear text displayed on combo-box Without using any private properties from the component. * refactor: move iteration props * fix: fix link to bceid * chore: remove unused import * fix: remove placeholders on country and province * fix: fix casing for Business information * fix: fix doc coment closing For some reason this was messing the code displayed in the Chrome browser sources. * fix: conditions used on postalCode validations * chore: fix VS Code resolution for source and test files By fixing the test configuration and eliminating a conflict with the app one. * test: use a new query command after a click Since it's unsafe to chain after a click. * test: force click button * fix: prevent completing business step with invalid data * fix: prevent fetching for name "undefined" * test: wait before clicking the option * Made a minor change for internal emails after a conversation with Andrew. * style: add semi-colon as requested after review * style: add the dollar signal to all conditions * feat: turn the dollar signal required * Made code reviews * Made more code reviews * Made this to prevent perisisting information mutliple times * test: wait for the option's inner element to become visible * fix: emit error always for text-input Instead of only when the error changes. * fix: update error status on Address step text fields So to be able to properly make it valid later when it gets resolved. * fix: update Street address empty status properly * test: emit empty true when contents is empty * docs: refer specific Jira issue * fix: - Renamed variable and added an new property to make it more generic - Beautified code * fix: - Renamed variable and added an new property in test file * fix: Made code reviews and removed unneeded prop I added * fix: update error status on Contacts step text fields So to be able to properly make it valid later when it gets resolved. * test: emit valid events properly * no message * fix: - Renamed variable with a proper name - Made code reviews * docs: improve comment * docs: improve comment * fix: synchronize dropdown value * fix: get country validation * fix: validate after changed from outside * test: define some tests to be implemented * test: province gets cleared * docs: add issue references * test: revalidates when changed from outside Also fixes another test. * Fast commit to switch branches * fix: validate the item code instead of its name * test: update tests * fix: remove meaningless tag * fix: do not display field as invalid When the field is empty and user didn't have a chance to add its value yet. For example, when it gets auto-changed as a result of changing another field. * Fast commit to switch branches * fix: Made code reviews and added birth date field * fix: Fixed logo for small screens * test: do not emit error in specific situation * feat: use variable to tell if selection was a user event * test: update tests * chore: use variable to tell if the change was a user event * test: update test * feat: add tooltip to button Next * fix: proceed applying part to next level Regardless of current element already having a part attribute. * chore: adjust combo-boxes focus outline * Fast commit to switch branches * Modified gitignore * feat: - Removed unused code - Made code reviews * feat: - Made more code reviews * Fast commit to switch branches * Renamed class with a more generic name * Fast commit to switch branches * Made more code reviews * Fast commit to switch branches * Pushed common functions * Fast commit to switch branches * fix: fix AutoComplete events - fix empty value; - always emit the error; - do not validate at each key pressed. * test: add empty and error tests * fix: fix initial empty value * removed unneeded key * fix: allow selecting item identical to the typed value * test: update test * Fast commit to switch branches * test: add address group test * Fast commit to switch branches * test: fix import * feat: show loading icon on auto-complete fields * feat: allow some time before showing the loading icon * chore: add helper text * fix: validate multi-select field * fix: remove deleted address from contacts * Fast commit to switch branches * chore: small fix to remove extra spaces on the first step * - Made code reviews - Improve code and made it more genetic * Made code reviews * FSADT1-928 * Fixed code smell issues * Fixed code smell issues * Create unit test * chore: forcing update * Changed annotation * Renamed file as it had a typo * chore: apply custom directives to unit tests * Fixed some smell issues * test: attempt to fix test on github actions * Made more code reviews * Made more code reviews * chore: adjust Application submitted breakpoints * chore: adjust margins on Application submitted * fix: adjust header buttons breakpoint Import missing variable. * Added more coverage * Beautified CSS file --------- Co-authored-by: Fernando Terra <[email protected]> Co-authored-by: Maria Martinez <[email protected]> Co-authored-by: Maria Martinez <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This is the second round of fixes for the external form.
Type of change
How Has This Been Tested?
Checklist
Further comments
Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Legacy available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow