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

Release v4.3.0 #380

Merged
merged 17 commits into from
Dec 11, 2024
Merged

Release v4.3.0 #380

merged 17 commits into from
Dec 11, 2024

Conversation

Adam-it
Copy link
Member

@Adam-it Adam-it commented Dec 10, 2024

🎯 Aim

The aim is to create a new minor release that adds more manage capabilities to the SPFx solutions view for every app catalog. It also includes a new /manage GitHub Copilot Chat command that allows to retrieve data from SharePoint Online. Improvements to the upgrade and validate actions which now allow to generate a code tour with guidance and added support for SPFx Fast Serve as an optional dependency.

📷 Result

spfx-chat2

sign-in-cache

spfx-fast-serve

app-catalog-list

✅ What was done

  • Added remove confirmation prompt when removing SPFx solution from app catalog
  • Added /manage GitHub Chat command
  • Added generating code tour for upgrade and validate SPFx solution actions
  • Added upgrade SPFx project action
  • Added SPFx Fast Serve optional dependency to the scaffolding form
  • Improved sign in experience with caching of Client ID and Tenant ID

@Adam-it Adam-it self-assigned this Dec 10, 2024
Saurabh7019 and others added 16 commits December 11, 2024 00:14
## 🎯 Aim

> The EntraApplicationCheck was previously used during the transition
period to verify usage of the PnP Management Shell app. Since the app
has been removed and the check is no longer needed, the
EntraApplicationCheck class and its reference in CommandPanel have been
removed.

## ✅ What was done

- [X] Removed EntraApplicationCheck class and its reference in
CommandPanel.

## 🔗 Related issue

Closes: #330
…loses #255, #331 (#341)

## 🎯 Aim

The aim of this PR is to:

- improve the general chat prompt to guide the user to the use the correct
chat command for the user intent
- add a new `/manage` command that will allow to retrieve data from
SharePoint

## 📷 Result

https://github.com/user-attachments/assets/7a1f1994-95c9-45ca-9015-94fb1a58c715

https://github.com/user-attachments/assets/24a722a7-3770-4252-be14-2a63f818c7d4

## ✅ What was done

- [x] improved general chat prompt
- [x] added manage chat command 
- [x] added script to parse CLI commands to create grounding data for
list and get commands for LLM
- [x] added separate LLM request to explain the CLI response 
- [x] improved transfer to `/code` chat command to include previous
history context

## 🔗 Related issue

Closes #255, #331
## 🎯 Aim

Is to do a small fixup in the way `/manage` chat command behaves when we are not logged in. It should break any other response until user will sign in.

## 📷 Result

![image](https://github.com/user-attachments/assets/cb1ca8cd-3992-4aa4-b9d8-18ed54929ce5)

## ✅ What was done

- [X] added return
- [X] small fixup in grounding file naming

## 🔗 Related issue

Closes: #345
## 🎯 Aim

Adding the tour options and both md & tour options as settings for the
upgrade and validate actions

## 📷 Result

![image](https://github.com/user-attachments/assets/582ac622-aa94-43f5-a3ae-644d24d36f17)

## ✅ What was done

- [X] Added `Md`, `Tour`, `Both` as upgrade options in the settings 
- [X] Added `Md`, `Tour`, `Both` as validate options in the settings 
- [X] Added tour logic to code for the upgrade option
- [X] Added tour logic to code for the validate option

## 🔗 Related issue

Closes: #188
## 🎯 Aim

Prompts for a confirmation before removing an app from the app catalog to prevent accidental removals and improve user experience.

## 📷 Result

![image](https://github.com/user-attachments/assets/967a48f7-a73c-457e-8dd1-6f61917b915b)

## ✅ What was done

- [X] Updated `removeAppCatalogApp` function to check the user's
response and proceed with removal only if confirmed.

## 🔗 Related issue

Closes: #349
…loses: #351 (#357)

## 🎯 Aim

Upgrades an app to the latest version available in the app catalog for the specified site

## 📷 Result

![image](https://github.com/user-attachments/assets/3c9255b0-28de-4365-9bce-d12452c611be)


## ✅ What was done

- [X] New action to upgrade an app in the specified site.
- [X] Prompt for the site URL only when upgrading tenant app catalog
apps. For site collection app catalog apps, it uses the site collection
app catalog URL and scope.

## 🔗 Related issue

Closes: #351
## 🎯 Aim

Improves upgrade app action by allowing users to specify only the
relative path and then concatenate it under the hood with the tenant URL

## 📷 Result

![image](https://github.com/user-attachments/assets/6699f07d-ce5c-45d0-bfaf-5dd26e744bac)

## ✅ What was done

[X] Prompt for the relative site URL when upgrading tenant app catalog
apps. Concatenate the same under the hood with the tenant URL.

## 🔗 Related issue

Closes: #359
## 🎯 Aim

The aim is to improve and simplify login behavior for SPFx Toolkit and try to
fixup a strange issue with login output when cli command is executed.

## ✅ What was done

- [x] updated login method from device code to browser to simplify the
process
- [x] added saving clientId and tenantId to the global state

## 🔗 Related issue

Closes #342, #374
… M365 commands. Closes: #375 (#378)

## 🎯 Aim

The aim is to update the command output mode to default to `text` for
`list` CLI for Microsoft 365 commands and `md` for `get` commands

## ✅ What was done

- [X] updated manage command grouding prompt
- [X] updated the output mode to use text for list commands

## 🔗 Related issue

Closes: #375
## 🎯 Aim

The aim is to add SPFx fast serve support to project scaffolding so that
it may be selected in as and optional dependency to install and
configure

## ✅ What was done

- [X] adds spfx-fast-serve optional step

## 🔗 Related issue

Closes: #218
Copy link
Member Author

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

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

I should update also the walkthrough images and text to align with latest changes

@Adam-it Adam-it marked this pull request as draft December 10, 2024 23:18
@Adam-it Adam-it marked this pull request as ready for review December 11, 2024 19:49
Copy link
Member Author

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

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

LGTM

@Adam-it Adam-it merged commit beaf740 into main Dec 11, 2024
3 checks passed
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