Updating SDK to use version latest minor version with 5.* by xShinnRyuu · Pull Request #26 · tableau/tableau-migration-app

xShinnRyuu · 2025-05-30T20:05:59Z

Updating SDK version to use 5.* instead of having to specify minor version so that changes in the Tableau server will have a smaller chance of breaking the Migration app with potentially breaking API changes.

GUI lib - Setup with Avalonia w/ MVVM template Tests - Setup with xUnit Console - Setup with CommandLineParser Added Dockerfile

gitignore file is updated for the visual studio setup

TMAPP-14

… project

…ption entites

[MigrationApp.Core] - small type fix

[MigrationApp.GUI] - Handled Dependency injections

…ion process

…puts

TMAPP-22 Fix Env Var typo and PlanBuilder auth type (#9)

TMAPP-23 Token fields as passwords

* Email domain mapping classed are added. Proper DI calls has been added. Plan is created using hook * IOptions is used instead of IOptions IMigrationPlanOptionsProvider which is used in the example EmailDomainMappingOptions is registered as a transient service populated in UI app * Removing duplicate imports * EmailDomainMappingOptions is updated in MainWindowViewModel * Unit Tests have been added * removed unused imports * renaming variable * {Core} Add test cases for email domain mapping: handle both existing and non-existing email scenarios * [Core] - Add null checks and assertions for potentially null variables --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

* TMAPP-31 Add Progress Bar for visual feedback of Migraiton Status Also includes TMAPP-24; the GUI definitions for the progress bar

* Email domain mapping classed are added. Proper DI calls has been added. Plan is created using hook * IOptions is used instead of IOptions IMigrationPlanOptionsProvider which is used in the example EmailDomainMappingOptions is registered as a transient service populated in UI app * Removing duplicate imports * EmailDomainMappingOptions is updated in MainWindowViewModel * Unit Tests have been added * Initial tool tips are added * removed unused imports * renaming variable * UserControl is limited with the help button * Help button added. Details link feature has been added * All help buttons are placed with dummy help texts * Help button onclick event capture is changed to PointerPressed from PointerReleased * Slight change on orientation of help button * Help button positon and design has been altered * Help button positon and design has been altered * The watermark of the Tableau Cloud URI has been altered --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

* Resize cancelled. Window will be resized accordingto the content automatically * Line endings normalized --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

* Initial docs setup * Added GitHub release workflow

…me (#26) * [GUI] - URIs and site names are parsed from the full URI * [GUI] - Enhanced parsing of server and cloud URIs to extract base URI and site name. Improved UI layout to better define sections and enhance user experience. --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

- Added all missing docstrings

- Removed/fixed small issues noticed during repo migration prep

* [GUI] - Add proper indentation to SplitButton * [GUI] - Bind OnClick events for Run and Resume Migration in SplitButton * [GUI] - Run and resume migration are arranged for split button * [GUI] - Refactor SplitButton properties for generic command and text binding * [GUI] - Add notification for missing manifest file on resume migration * [GUI] - Define RunMigrationCommand IAsyncRelayCommand instead of ICommand * [GUI] - Remove notification SelectManifestFileAsync returns null, add logs instead * [GUI] - Use RelayCommand for public accessibility of RunMigrationCommand and ResumeMigrationCommand properties --------- Co-authored-by: Lester Szeto <Lester.Szeto@improving.com> Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

…ate successful clicks on the link (#51) * [GUI] - Add styling to help button for responsiveness of the links * [GUI] Directly bind Click event in HelpButton XAML, use native button for better onpressed behaviour. --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

* tmapp-71 Usage and Development docs

* SDK Version Update + Name Fixes * Updated Core Tests Logging version

* [Core][GUI] - Refactored progress message handling with elapsed time tracking * [Core] - Update summries of methods in IProgressMessagePublisher * Sytlecop fix * [GUI] Create timerDisplay user control and view model * [GUI] - Add ProgressTimerController with UI updates for real-time migration and action timers * [GUI] - Add unit tests for TimerDisplayViewModel * [GUI] - Add Unit tests for ProgressTimerController * [Core][GUI] - Clean up * [GUI] - Small refactoring * [GUI] - Reset action and total migration timers after completion * [GUI] - Adjust timer test accordingly --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

…ion buttons (#56) * [GUI] - Add HelpText support to SplitButton for Run and Resume Migration buttons * [GUI] - Add short circuit return to SaveManifestIfRequiredAsync function * [GUI] - Remove HelpButton from SplitButton, add it to MainWindow instaed --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

… prompted (#57)

* TMAPP-80 bug fix: Progress Bar status is offset * Timing state stored - Reverted buggy implementation - Created Timing service - Set timer states based on progress state progression - Set MessageDisplayVM to be DI injected - Added new MigrationTimer class - Added TimersView and TimersViewModel to hold migration timing - Refactored message publishing logic * Changed timer labels to action names

* Added Coverage report generation - Added tests to increase coverage - Added gen-report.sh to generate combined coverage report

* Fixed tests and GH workflow

* TMAPP-79 Various GUI adjustments - Removed horizontal line above "Run Migration" button - Fixed spacing between User Configuration labels and textboxes - Set UserFileMappingsViewModel to use the same validation error logic as the other components - Added spacing around progress bar

* TMAPP-82 Default saving manifest after migration

TMAPP 81, 92, 93 * TMAPP-81 Fix MacOs Top Bar Menu * TMAPP-92 Optional Mappings * TMAPP-93 Refactoring and more tests

salesforce-cla · 2025-05-30T20:06:03Z

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Muzaffer Aydin <M***@i***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

salesforce-cla · 2025-05-30T20:06:03Z

Thanks for the contribution! Before we can merge this, we need @muaydin to sign the Salesforce Inc. Contributor License Agreement.

github-actions · 2025-05-30T20:06:58Z

Code Coverage Summary

Summary
  Generated on: 05/30/2025 - 20:06:58
  Parser: MultiReport (2x OpenCover)
  Assemblies: 2
  Classes: 52
  Files: 63
  Line coverage: 77.2%
  Covered lines: 1173
  Uncovered lines: 346
  Coverable lines: 1519
  Total lines: 5323
  Branch coverage: 56.8% (244 of 429)
  Covered branches: 244
  Total branches: 429
  Method coverage: 84.8% (229 of 270)
  Full method coverage: 77.7% (210 of 270)
  Covered methods: 229
  Fully covered methods: 210

lszinv and others added 30 commits August 22, 2024 15:09

Initial Commit - Project framework setup

892e456

GUI lib - Setup with Avalonia w/ MVVM template Tests - Setup with xUnit Console - Setup with CommandLineParser Added Dockerfile

[EsmbMigration.GUI] - Project prototype has been added

301d80f

gitignore file is updated for the visual studio setup

[EsmbMigrationApp.GUI] - Tableau.Migration NuGet package has been added

a1de20c

Adding Github Action unit test hook to PRs to main

592b056

Per discussion with Tableau, removing Emsb from application name

c03a7da

TMAPP-14

Initial folder structure for .Core project

1e0b137

Temp class has been removed

5ff8acd

[MigrationApp.Core] - Project architecture has been created for .Core…

b8ad8b5

… project

TMAPP-11 Added test to verify connection with SDK and Server / Cloud

7b05426

[MigrationApp.GUI] - namespace change

a2f3a84

[MigrationApp.GUI] - UI elements are arranged according to Endpoint o…

4e4c5dd

…ption entites

[MigrationApp.GUI] - Error validation has been aded for URI inputs

4ab9135

[MigrationApp.GUI] - .Core DLL integration is completed

95d6aa6

[MigrationApp.Core] - small type fix

[MigrationApp.Core] - ILogger template type fix

9020b3d

[MigrationApp.GUI] - Handled Dependency injections

[MigrationApp.GUI] - Notification mechanism has been added for migrat…

41a8d50

…ion process

Remove empty validation for ServerSiteContent and CloudSiteContent in…

4e13522

…puts

TMAPP-22 Fix Env Var typo and PlanBuilder auth type (#9)

08c00d6

TMAPP-16 Config loading for Core module (#8)

675c01a

TMAPP-22 Fix Env Var typo and PlanBuilder auth type (#9)

TMAPP-23 [GUI] Access Token fields to Password (#10)

fdcbe7e

TMAPP-23 Token fields as passwords

TMAPP-32 Fix Build Warnings

f310331

TMAPP-28 Disable fields while migrating (#11)

440cd8b

TMAPP-30 Defining editor style rules (#12)

f1e2987

TMAPP-31 Add Progress Bar for visual feedback of Migraiton Status (#18)

9e4362a

* TMAPP-31 Add Progress Bar for visual feedback of Migraiton Status Also includes TMAPP-24; the GUI definitions for the progress bar

Tmapp 33 GUI fix app resizing visual issues (#19)

9b96d2a

* Resize cancelled. Window will be resized accordingto the content automatically * Line endings normalized --------- Co-authored-by: Muzaffer Aydin <Muzaffer.Aydin@improving.com>

Generate documentation and release packages (#22)

54a5d5a

* Initial docs setup * Added GitHub release workflow

TMAPP-43 Add Stylecop formatting enforcement

7215cd4

- Added all missing docstrings

[Core] Setting UserAgent suffix

1beaecf

lszinv and others added 24 commits October 22, 2024 17:08

Additional cleanup (#46)

c19451a

- Removed/fixed small issues noticed during repo migration prep

Fixed README Typos (#48)

5c739f1

TMAPP-61 Scripts to build MacOS Packages (#49)

d0eaea3

tmapp-71 Usage and Development docs (#50)

9f98fd6

* tmapp-71 Usage and Development docs

Workflow with MacApp Releases (#52)

2dfa443

Readme Update (#55)

49cae58

SDK Version Update + Name Fixes (#54)

5f218b2

* SDK Version Update + Name Fixes * Updated Core Tests Logging version

Tmapp 85 Cancel migration should cancel before the manifest saving is…

a448b5b

… prompted (#57)

Added Coverage report generation (#58)

1de892d

* Added Coverage report generation - Added tests to increase coverage - Added gen-report.sh to generate combined coverage report

Fixing test (#65)

fffb3a1

* Fixed tests and GH workflow

TMAPP-87 Window Scaling (#63)

9528355

TMAPP-82 Default saving manifest after migration (#64)

8f7148a

* TMAPP-82 Default saving manifest after migration

TMAPP-93 View Constants (#66)

3ed37e6

Tmapp 92 - Optional User Mappings (#67)

ebb87c5

TMAPP 81, 92, 93 * TMAPP-81 Fix MacOs Top Bar Menu * TMAPP-92 Optional Mappings * TMAPP-93 Refactoring and more tests

doc fixes (#68)

e29b2fa

Merge pull request from public Tableau Migration App repo

5b502d1

Merge remote-tracking branch 'upstream/main'

ccf94ef

Updating SDK to use version latest minor version with 5.*

8348573

salesforce-cla Bot added the cla:missing label May 30, 2025

xShinnRyuu closed this May 30, 2025

xShinnRyuu deleted the update_SDK_to_5.2 branch May 30, 2025 20:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Updating SDK to use version latest minor version with 5.*#26

Updating SDK to use version latest minor version with 5.*#26
xShinnRyuu wants to merge 75 commits into
mainfrom
update_SDK_to_5.2

xShinnRyuu commented May 30, 2025

Uh oh!

salesforce-cla Bot commented May 30, 2025

Uh oh!

salesforce-cla Bot commented May 30, 2025

Uh oh!

github-actions Bot commented May 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

xShinnRyuu commented May 30, 2025

Uh oh!

salesforce-cla Bot commented May 30, 2025

Uh oh!

salesforce-cla Bot commented May 30, 2025

Uh oh!

github-actions Bot commented May 30, 2025

Code Coverage Summary

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants