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

feat(hub-common): disable categories field and provide notice when no options are available #1561

Merged
merged 9 commits into from
Jun 24, 2024

Conversation

sonofflynn89
Copy link
Contributor

@sonofflynn89 sonofflynn89 commented Jun 21, 2024

  1. Description:
    Part of https://devtopia.esri.com/dc/hub/issues/10382

Highlights:

  • Abstracts the creation of the categories ui schema field into a helper function to maintain consistency.
  • Adds a notice to all categories schema fields when no categories are available within the user's org
  • Adds a link configuration object to IUiSchemaMessage that can only be used when rendering notices
  1. Updated meaningful TSDoc to methods including Parameters and Returns, see Documentation Guide

  2. used semantic commit messages

  3. PR title follows semantic commit format (CRITICAL if the title is not in a semantic format, the release automation will not run!)

…ent()" helper to prevent duplic

affects: @esri/hub-common
…categories are available

affects: @esri/hub-common
@sonofflynn89 sonofflynn89 changed the title B/10382 fix categories UI schema field feat(hub-common): disable categories field and provide notice when no options are available Jun 21, 2024
…nk instead of inlined type

affects: @esri/hub-common
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (7698062) to head (ac09640).
Report is 18 commits behind head on master.

Current head ac09640 differs from pull request most recent head 848d47d

Please upload reports for the commit 848d47d to get more accurate results.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #1561   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          963       966    +3     
  Lines        17377     17449   +72     
  Branches      3027      3043   +16     
=========================================
+ Hits         17377     17449   +72     

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

Copy link
Contributor

@abp6318 abp6318 left a comment

Choose a reason for hiding this comment

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

I agreed with the comments that Julianne left, and tested things locally (including some a11y checks). Looks good! I left a comment on the other PR about repeated strings that we might be able to reduce before you merge this.

Copy link
Contributor

@juliannemarik juliannemarik left a comment

Choose a reason for hiding this comment

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

This all looks good to me as well - thanks for making the updates and for the addition to the interface!

…s should always use 'launch')

affects: @esri/hub-common
@sonofflynn89 sonofflynn89 merged commit 95035c0 into master Jun 24, 2024
5 checks passed
@sonofflynn89 sonofflynn89 deleted the b/10382-fix-categories-ui-schema-field branch June 24, 2024 18:55
esri-dcdev-2 pushed a commit that referenced this pull request Jun 24, 2024
# @esri/hub-common [14.150.0](https://github.com/Esri/hub.js/compare/@esri/hub-common@14.149.0...@esri/hub-common@14.150.0) (2024-06-24)

### Features

* **hub-common:** disable categories field and provide notice when no options are available ([#1561](#1561)) ([95035c0](95035c0))
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.

None yet

3 participants