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

[20/X] DXCDT-441: Reintroduce support for zoom client addon #677

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

sergiught
Copy link
Contributor

🔧 Changes

Reintroducing support for zoom client addon on the v1 branch after correctly typing them to the new go-auth0 types.

📚 References

🔬 Testing

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@sergiught sergiught requested a review from a team as a code owner June 29, 2023 08:42
@codecov-commenter
Copy link

Codecov Report

Merging #677 (b89d0e5) into feature/DXCDT-441-client-addons-zendesk-19 (993f945) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@                              Coverage Diff                               @@
##           feature/DXCDT-441-client-addons-zendesk-19     #677      +/-   ##
==============================================================================
+ Coverage                                       87.68%   87.71%   +0.03%     
==============================================================================
  Files                                              80       80              
  Lines                                           12859    12896      +37     
==============================================================================
+ Hits                                            11275    11312      +37     
  Misses                                           1211     1211              
  Partials                                          373      373              
Impacted Files Coverage Δ
internal/auth0/client/expand.go 94.72% <100.00%> (+0.12%) ⬆️
internal/auth0/client/flatten.go 97.95% <100.00%> (+0.05%) ⬆️
internal/auth0/client/resource.go 98.06% <100.00%> (+0.02%) ⬆️

Elem: &schema.Resource{
Schema: map[string]*schema.Schema{
"account": {
Description: "Zoom account name usually first segment of your Zoom URL, for " +
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Description: "Zoom account name usually first segment of your Zoom URL, for " +
Description: "Zoom account name. Usually the first segment of your Zoom URL, for " +

@sergiught sergiught force-pushed the feature/DXCDT-441-client-addons-zendesk-19 branch 2 times, most recently from d43a41c to ac73fd7 Compare June 29, 2023 15:50
@sergiught sergiught force-pushed the feature/DXCDT-441-client-addons-zoom-20 branch from b89d0e5 to 4bd4346 Compare June 29, 2023 15:55
@sergiught sergiught requested a review from Widcket June 29, 2023 16:00
Base automatically changed from feature/DXCDT-441-client-addons-zendesk-19 to v1 June 29, 2023 16:05
@sergiught sergiught force-pushed the feature/DXCDT-441-client-addons-zoom-20 branch from 4bd4346 to 6d3200d Compare June 29, 2023 16:06
@sergiught sergiught merged commit 99395dd into v1 Jun 29, 2023
@sergiught sergiught deleted the feature/DXCDT-441-client-addons-zoom-20 branch June 29, 2023 16:12
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