Hotfix schedule_item.py for issue 1237 #1239
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TableauIDWithMFA added to the user_item model to allow creating users on Tableau Cloud with MFA enabled (tableau#1217) Run long requests on second thread (tableau#1212) tableau#1210 tableau#1087 tableau#1058 tableau#456 tableau#1209 update datasource to use bridge (tableau#1224) Co-authored-by: Tim Payne <47423639+ma7tcsp@users.noreply.github.com>
|
Thanks for the contribution! Before we can merge this, we need @austinpeters-gohealthuccom to sign the Salesforce Inc. Contributor License Agreement. |
|
|
jacalata
approved these changes
Jun 9, 2023
|
@austinpeters-gohealthuccom are you able to reformat this for our style checker? |
|
I literally commented out 3 lines and that's it. Means whoever committed the last branch got a bypass. |
|
Think it just wants spaces between the # and the commented code? |
Spaces in comments? why not
|
Hello, I stumbled on this exact issue and I was wondering if there is any scheduled release/merge of this fix? |
jacalata
pushed a commit
that referenced
this pull request
Aug 1, 2023
Co-authored-by: Tim Payne <47423639+ma7tcsp@users.noreply.github.com> Hotfix for #1237
jacalata
added a commit
that referenced
this pull request
Aug 1, 2023
* Hotfix schedule_item.py for issue 1237 (#1239) * Remove duplicate assignments to fields (#1244) Co-authored-by: Tim Payne <47423639+ma7tcsp@users.noreply.github.com> Co-authored-by: Austin <110413815+austinpeters-gohealthuccom@users.noreply.github.com> Co-authored-by: Yasuhisa Yoshida <syou6162@gmail.com>
jacalata
added a commit
that referenced
this pull request
Sep 22, 2023
* fix: TableauIDWithMFA added to the user_item model to allow creating users on Tableau Cloud with MFA enabled (#1216) * fix: make project optional in datasources #1210 * fix: allow setting timeout on workbook endpoint #1087 * fix: can't certify datasource on publish #1058 * fix filter in operator spaces bug (#1259) * fix: remove logging configuration from TSC (#1248) * Hotfix schedule_item.py for issue 1237 (#1239), Remove duplicate assignments to fields (#1244) * Fix shared attribute for custom views (#1280) New functionality * enable filtering for Excel downloads #1209, #1281 * query view by content url #456 * update datasource to use bridge (#1224) * Add JWTAuth, add repr using qualname * Add publish samples attribute (#1264) * add support for custom schedules in TOL (#1273) * Enable asJob for group update (#1276) Co-authored-by: Tim Payne <47423639+ma7tcsp@users.noreply.github.com> Co-authored-by: Lars Breddemann <139097050+LarsBreddemann@users.noreply.github.com> Co-authored-by: jorwoods <jorwoods@users.noreply.github.com> Co-authored-by: Austin <110413815+austinpeters-gohealthuccom@users.noreply.github.com> Co-authored-by: Yasuhisa Yoshida <syou6162@gmail.com> Co-authored-by: Brian Cantoni <bcantoni@salesforce.com> Co-authored-by: a-torres-2 <142839181+a-torres-2@users.noreply.github.com> Co-authored-by: Łukasz Włodarczyk <lukasz@pushmetrics.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hotfix for #1237