Skip to content

Conversation

@Rick-Kirkham
Copy link
Contributor

Change Description:

Support multiword add-in names

  1. Do these changes impact any npm scripts commands (in package.json)? (e.g., running 'npm run start')
    If Yes, briefly describe what is impacted.
    no

  2. Do these changes impact VS Code debugging options (launch.json)?
    If Yes, briefly describe what is impacted.
    no

  3. Do these changes impact template output? (e.g., add/remove file, update file location, update file contents)
    If Yes, briefly describe what is impacted.
    yes, fixes manifest

  4. Do these changes impact documentation? (e.g., a tutorial on https://docs.microsoft.com/en-us/office/dev/add-ins/overview/office-add-ins)
    If Yes, briefly describe what is impacted.
    no

If you answered yes to any of these please do the following:
> Include 'Rick-Kirkham' in the review
> Make sure the README file is correct

Validation/testing performed:

Tested locally with multiword names. Works as desired.

@Rick-Kirkham Rick-Kirkham requested a review from a team as a code owner April 25, 2024 23:50
@Rick-Kirkham Rick-Kirkham merged commit bcc4013 into master Apr 26, 2024
@Rick-Kirkham Rick-Kirkham deleted the multiword-names branch April 29, 2024 21:22
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