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

Remove deprecated methods marked for removal in 4.0 #83

Open
wants to merge 1 commit into
base: mainline
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view

This file was deleted.

This file was deleted.

55 changes: 0 additions & 55 deletions src/main/java/com/smartsheet/api/AttachmentResources.java

This file was deleted.

46 changes: 0 additions & 46 deletions src/main/java/com/smartsheet/api/ColumnResources.java

This file was deleted.

50 changes: 0 additions & 50 deletions src/main/java/com/smartsheet/api/CommentResources.java

This file was deleted.

7 changes: 0 additions & 7 deletions src/main/java/com/smartsheet/api/DiscussionResources.java
Original file line number Diff line number Diff line change
Expand Up @@ -42,13 +42,6 @@ public interface DiscussionResources {
*/
Comment addDiscussionComment(long id, Comment comment) throws SmartsheetException;

/**
* @return associated resources
* @deprecated As of release 2.0. Please use the corresponding method in the [Row/Sheet/etc]AttachmentResources classes
*/
@Deprecated(since = "2.0.0", forRemoval = true)
AssociatedAttachmentResources attachments();

/**
* <p>Represents the DiscussionCommentResources.</p>
* <p>It will be initialized in constructor and will not change afterwards.</p>
Expand Down
21 changes: 0 additions & 21 deletions src/main/java/com/smartsheet/api/ShareResources.java
Original file line number Diff line number Diff line change
Expand Up @@ -29,27 +29,6 @@
*/
public interface ShareResources {

/**
* <p>List shares of a given object.</p>
*
* <p>It mirrors to the following Smartsheet REST API method:</p>
* <p> GET /workspace/{id}/shares </p>
* <p> GET /sheet/{id}/shares</p>
*
* @param objectId the object id
* @param parameters the pagination parameters
* @return the list of Share objects (note that an empty list will be returned if there is none).
* @throws IllegalArgumentException if any argument is null or empty string
* @throws InvalidRequestException if there is any problem with the REST API request
* @throws AuthorizationException if there is any problem with the REST API authorization (access token)
* @throws ResourceNotFoundException if the resource cannot be found
* @throws ServiceUnavailableException if the REST API service is not available (possibly due to rate limiting)
* @throws SmartsheetException if there is any other error during the operation
* @deprecated As of release 2.0. Please use the other listShares method in this class and pass `includeWorkspaceShares` as `false`
*/
@Deprecated(since = "2.0.0", forRemoval = true)
PagedResult<Share> listShares(long objectId, PaginationParameters parameters) throws SmartsheetException;

/**
* <p>List shares of a given object.</p>
*
Expand Down
42 changes: 0 additions & 42 deletions src/main/java/com/smartsheet/api/SheetResources.java
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,12 @@
package com.smartsheet.api;

import com.smartsheet.api.models.ContainerDestination;
import com.smartsheet.api.models.MultiRowEmail;
import com.smartsheet.api.models.PagedResult;
import com.smartsheet.api.models.PaginationParameters;
import com.smartsheet.api.models.Sheet;
import com.smartsheet.api.models.SheetEmail;
import com.smartsheet.api.models.SheetPublish;
import com.smartsheet.api.models.SortSpecifier;
import com.smartsheet.api.models.UpdateRequest;
import com.smartsheet.api.models.enums.CopyExclusion;
import com.smartsheet.api.models.enums.ObjectExclusion;
import com.smartsheet.api.models.enums.PaperSize;
Expand Down Expand Up @@ -84,24 +82,6 @@ PagedResult<Sheet> listSheets(
*/
PagedResult<Sheet> listSheets(EnumSet<SourceInclusion> includes, PaginationParameters pagination) throws SmartsheetException;

/**
* <p>List all sheets in the organization.</p>
*
* <p>It mirrors to the following Smartsheet REST API method: GET /users/sheets</p>
*
* @param parameters the object containing the pagination parameters
* @return the list of all sheets (note that an empty list will be returned if there are none)
* @throws IllegalArgumentException if any argument is null or empty string
* @throws InvalidRequestException if there is any problem with the REST API request
* @throws AuthorizationException if there is any problem with the REST API authorization (access token)
* @throws ResourceNotFoundException if the resource cannot be found
* @throws ServiceUnavailableException if the REST API service is not available (possibly due to rate limiting)
* @throws SmartsheetException if there is any other error during the operation
* @deprecated As of release 2.0. Please use {@link UserResources} instead
*/
@Deprecated(since = "2.0.0", forRemoval = true)
PagedResult<Sheet> listOrganizationSheets(PaginationParameters parameters) throws SmartsheetException;

/**
* <p>Get a sheet.</p>
*
Expand Down Expand Up @@ -671,28 +651,6 @@ Sheet copySheet(long sheetId, ContainerDestination containerDestination, EnumSet
*/
Sheet moveSheet(long sheetId, ContainerDestination containerDestination) throws SmartsheetException;

/**
* <p>Creates an Update Request for the specified Row(s) within the Sheet.</p>
*
* <p>It mirrors to the following Smartsheet REST API method: POST /sheets/{sheetId}/updaterequests</p>
*
* Exceptions:
* - IllegalArgumentException : if any argument is null
* - InvalidRequestException : if there is any problem with the REST API request
* - AuthorizationException : if there is any problem with the REST API authorization(access token)
* - ServiceUnavailableException : if the REST API service is not available (possibly due to rate limiting)
* - SmartsheetRestException : if there is any other REST API related error occurred during the operation
* - SmartsheetException : if there is any other error occurred during the operation
*
* @param sheetId the sheet id
* @param email the email
* @return the update request object
* @throws SmartsheetException the smartsheet exception
* @deprecated As of release 2.0. Please use {@link SheetUpdateRequestResources} instead.
*/
@Deprecated(since = "2.0.0", forRemoval = true)
UpdateRequest createUpdateRequest(long sheetId, MultiRowEmail email) throws SmartsheetException;

/**
* <p>Sort a sheet according to the sort criteria.</p>
*
Expand Down
Loading
Loading