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(naming): dynamically generate unique name for auro-dropdown #127 #128

Closed
wants to merge 7 commits into from

Conversation

jason-capsule42
Copy link
Member

Alaska Airlines Pull Request

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes: #127

Summary:

This work shows using a dynamically generated hash to create a unique element name used during the custom element definition

Type of change:

Please delete options that are not relevant.

  • New capability
  • Revision of an existing capability
  • Infrastructure change (automation, etc.)
  • Other (please elaborate)

Checklist:

  • My update follows the CONTRIBUTING guidelines of this project
  • I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.

Thank you for your submission!

-- Auro Design System Team

@jason-capsule42 jason-capsule42 self-assigned this Apr 27, 2023
@jason-capsule42 jason-capsule42 requested a review from a team as a code owner April 27, 2023 17:00
Base automatically changed from jjones/swapValueExample/#125 to main May 2, 2023 20:07
@jordanjones243 jordanjones243 linked an issue May 10, 2023 that may be closed by this pull request
@blackfalcon
Copy link
Member

Please update commit a7bf217 to have notes that this repo was updated to Lit2.0 as well as create the new feature.

Also commits b8abdee and 4a130c8 can all be merged into the single a7bf217 commit. Those updates and a repeated feat(naming): improving dynamic naming of deps message has no real long-term historical value.

Copy link
Member

@blackfalcon blackfalcon left a comment

Choose a reason for hiding this comment

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

Please see other comments.

scripts/version.js Outdated Show resolved Hide resolved
Copy link
Member

@blackfalcon blackfalcon left a comment

Choose a reason for hiding this comment

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

@jason-capsule42
Copy link
Member Author

Closing pull request in favor of new attempt: #148

@jason-capsule42 jason-capsule42 deleted the jbaker/hashNamingPOC branch June 24, 2024 22:48
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.

Dynamic dependency CE naming POC
4 participants