Merge pull request #3198 from omnivore-app/fix/move-to-folder-api

fix moveToFolder API error by skipping fetching content again
This commit is contained in:
Hongbo Wu 2023-12-04 16:10:09 +08:00 committed by GitHub
commit 37beab64c6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 22 additions and 25 deletions

View file

@ -1301,7 +1301,7 @@ export type MoveToFolderResult = MoveToFolderError | MoveToFolderSuccess;
export type MoveToFolderSuccess = {
__typename?: 'MoveToFolderSuccess';
articleSavingRequest: ArticleSavingRequest;
success: Scalars['Boolean'];
};
export type Mutation = {
@ -5328,7 +5328,7 @@ export type MoveToFolderResultResolvers<ContextType = ResolverContext, ParentTyp
};
export type MoveToFolderSuccessResolvers<ContextType = ResolverContext, ParentType extends ResolversParentTypes['MoveToFolderSuccess'] = ResolversParentTypes['MoveToFolderSuccess']> = {
articleSavingRequest?: Resolver<ResolversTypes['ArticleSavingRequest'], ParentType, ContextType>;
success?: Resolver<ResolversTypes['Boolean'], ParentType, ContextType>;
__isTypeOf?: IsTypeOfResolverFn<ParentType, ContextType>;
};

View file

@ -1165,7 +1165,7 @@ enum MoveToFolderErrorCode {
union MoveToFolderResult = MoveToFolderError | MoveToFolderSuccess
type MoveToFolderSuccess {
articleSavingRequest: ArticleSavingRequest!
success: Boolean!
}
type Mutation {

View file

@ -916,25 +916,25 @@ export const moveToFolderResolver = authorized<
const savedAt = new Date()
// if the content is not fetched yet, create a page save request
if (!item.readableContent) {
const articleSavingRequest = await createPageSaveRequest({
userId: uid,
url: item.originalUrl,
articleSavingRequestId: id,
priority: 'high',
publishedAt: item.publishedAt || undefined,
savedAt,
pubsub,
})
// // if the content is not fetched yet, create a page save request
// if (!item.readableContent) {
// const articleSavingRequest = await createPageSaveRequest({
// userId: uid,
// url: item.originalUrl,
// articleSavingRequestId: id,
// priority: 'high',
// publishedAt: item.publishedAt || undefined,
// savedAt,
// pubsub,
// })
return {
__typename: 'MoveToFolderSuccess',
articleSavingRequest,
}
}
// return {
// __typename: 'MoveToFolderSuccess',
// articleSavingRequest,
// }
// }
const updatedItem = await updateLibraryItem(
await updateLibraryItem(
item.id,
{
folder,
@ -946,10 +946,7 @@ export const moveToFolderResolver = authorized<
return {
__typename: 'MoveToFolderSuccess',
articleSavingRequest: libraryItemToArticleSavingRequest(
updatedItem.user,
updatedItem
),
success: true,
}
})

View file

@ -2665,7 +2665,7 @@ const schema = gql`
union MoveToFolderResult = MoveToFolderSuccess | MoveToFolderError
type MoveToFolderSuccess {
articleSavingRequest: ArticleSavingRequest!
success: Boolean!
}
type MoveToFolderError {