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

add detail comment (part 2) #15

Merged
merged 3 commits into from
Feb 24, 2020
Merged

Conversation

severinbeauvais
Copy link
Collaborator

@severinbeauvais severinbeauvais commented Feb 20, 2020

Issue #: /bcgov/entity#2719

Description of changes:

  • added Add Comment Dialog unit tests
  • added Detail Comment unit tests
  • updated Staff Payment unit tests
  • refactored Directors unit tests to not use setTimeout()
  • fixed filing save tests (missing 'get tasks' stub)
  • cleaned up other unit tests
  • added Detail Comment autofocus prop
  • display app version on startup to console deleted
  • removed obsolete hard-coded entity type
  • ensured comments are sorted by timestamp
  • reload just the new comments, not the whole dashboard

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).

Copy link
Collaborator

@cameron-eyds cameron-eyds left a comment

Choose a reason for hiding this comment

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

Looks good, just had a few comments/questions, nothing that would block this tho! Good clean up in the unit tests.

Copy link
Collaborator

@thorwolpert thorwolpert left a comment

Choose a reason for hiding this comment

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

Styling: I notice you put your methods mostly on newlines, which I do as well. I found that easier to read than some of the other sections. What do you think about encouraging others to use that style?

src/utils/config-helper.ts Outdated Show resolved Hide resolved
- added Add Comment Dialog unit tests
- added Detail Comment unit tests
- refactored Directors unit tests to not use setTimeout()
- fixed filing save tests (missing 'get tasks')
- cleaned up other unit tests
- added Detail Comment autofocus prop
- display app version on startup to console
- removed obsolete hard-coded entity type (code still not in use)
- ensures comments are sorted by timestamp
@sonarcloud
Copy link

sonarcloud bot commented Feb 24, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 2 Code Smells

No Coverage information No Coverage information
18.0% 18.0% Duplication

@thorwolpert thorwolpert merged commit 83bfef5 into bcgov:master Feb 24, 2020
thorwolpert pushed a commit to thorwolpert/business-filings-ui that referenced this pull request Mar 16, 2020
* oracledb not starting

Signed-off-by: Kial Jinnah <kialj876@gmail.com>

* working dockerfile

Signed-off-by: Kial Jinnah <kialj876@gmail.com>

* oracle running in openshift

Signed-off-by: Kial Jinnah <kialj876@gmail.com>

* cleaned up dockerfile/echo stmnts

Signed-off-by: Kial Jinnah <kialj876@gmail.com>

* deployment config added

Signed-off-by: Kial Jinnah <kialj876@gmail.com>

* readme for oracle container

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
severinbeauvais pushed a commit to severinbeauvais/business-filings-ui that referenced this pull request Mar 19, 2021
completed files for build/deploy from GH Actions
severinbeauvais pushed a commit to severinbeauvais/business-filings-ui that referenced this pull request Mar 19, 2021
* initial commit

* Updated for initial PR

* Removed padding class

* Pr Comments Update

* Initial Commit. Form Implementation

* Wrap up NameRequest UI correction

* import clean up

* Console clean up

* Pr Updates
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.

3 participants