Project

General

Profile

Task #3229

Feature #3226: Support for copilot_urls

Feature #3227: BE : Support for copilot_urls

Return 'showCopilot' in the response of getPurpleLinksInfoExtFull API

Added by Anonymous over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Immediate
Assignee:
Target version:
Start date:
09/26/2023
Due date:
% Done:

100%

Estimated time:
1.50 h
Spent time:

Description

IF the "copilot_urls" field is present in the datastore, and the requested "o-url" is neither a prefix in any of the "copilot_urls" nor found within "copilot_urls," then set "showCopilot" to false.

ELSE
Set "showCopilot" as true for all the 3 existing cases - https://docs.google.com/document/d/1uMTr3gKDf_XxMZse9j0ys-XwTEzM32TJwDH31SAiVA0/edit#heading=h.28q2j3w7old3

Handle this for 0.95 and later versions

#1

Updated by Anonymous over 1 year ago

  • Assignee set to Anonymous
#2

Updated by Anonymous over 1 year ago

  • Description updated (diff)
#3

Updated by Anonymous over 1 year ago

  • Status changed from New to In Progress
#4

Updated by Anonymous over 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
#5

Updated by Anonymous over 1 year ago

  • Status changed from Resolved to Review
  • Assignee changed from Anonymous to Anonymous
#6

Updated by Anonymous over 1 year ago

  • Description updated (diff)
#7

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Anonymous
#8

Updated by Ram Kordale over 1 year ago

  • Priority changed from High to Immediate
#9

Updated by Anonymous over 1 year ago

  • Status changed from Review to Reopened
  • Assignee changed from Anonymous to Anonymous

Support for internal-ourls : The 'showCopilot' should be 'true' even if any of the "internal-ourls" is present in the "copilot_urls"

#10

Updated by Anonymous over 1 year ago

  • Status changed from Reopened to In Progress
#11

Updated by Anonymous over 1 year ago

  • Status changed from In Progress to Resolved
#12

Updated by Anonymous over 1 year ago

  • Status changed from Resolved to Review
  • Assignee changed from Anonymous to Anonymous
#13

Updated by Anonymous over 1 year ago

  • Status changed from Review to Reopened
  • Assignee changed from Anonymous to Anonymous
  • % Done changed from 100 to 50

Bug:
For all old books, show_copilot is coming as false. This is incorrect. If the `copilot_urls` field is not present in the text file, showCopilot should come as true for all urls of the book. so, it should be true for all the old books. Even for new books if the `copilot_urls` field is not present, showCopilot should be true for all urls of the book.

Add this fix - When `copilot_urls` field is not present in the datastore, set the showCopilot as true

#14

Updated by Anonymous over 1 year ago

  • Status changed from Reopened to In Progress
#15

Updated by Anonymous over 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100
#16

Updated by Anonymous over 1 year ago

  • Status changed from Resolved to Review
  • Assignee changed from Anonymous to Anonymous
#17

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Anonymous
#18

Updated by Anonymous over 1 year ago

  • Status changed from Review to Feedback
  • Assignee changed from Anonymous to Anonymous
#19

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Anonymous
#20

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Anonymous
#21

Updated by Ram Kordale over 1 year ago

  • Status changed from Feedback to Closed
#22

Updated by Ram Kordale over 1 year ago

  • Status changed from Closed to Reopened
  • Assignee changed from Anonymous to Anonymous
#23

Updated by Anonymous over 1 year ago

  • Status changed from Reopened to In Progress
#24

Updated by Anonymous over 1 year ago

  • Status changed from In Progress to Resolved
#25

Updated by Anonymous over 1 year ago

  • Status changed from Resolved to Review
  • Assignee changed from Anonymous to Anonymous
#26

Updated by Anonymous over 1 year ago

  • Description updated (diff)
  • Assignee changed from Anonymous to Anonymous
#27

Updated by Anonymous over 1 year ago

  • Description updated (diff)
#28

Updated by Anonymous over 1 year ago

  • Description updated (diff)

Support : Implement Copilot URL Wildcard Matching
For the given Copilot URL pattern: "copilot_urls": ["scikit-learn.org/*blog*"], (Note: blog could be any string)
If the copilot url contains * then check if it is matching with the requested Ourl.

If the requested OURL is http://scikit-learn.org/modulesblog/1/2/3 or https://scikit-learn.org/1/2/3/4blog5 the matching should result in "showCopilot": true.

#29

Updated by Anonymous over 1 year ago

  • Status changed from Review to Feedback
  • Assignee changed from Anonymous to Anonymous
#30

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Anonymous
#31

Updated by Anonymous over 1 year ago

  • Assignee changed from Anonymous to Ram Kordale
#32

Updated by Ram Kordale over 1 year ago

  • Priority changed from Immediate to Urgent
#33

Updated by Ram Kordale over 1 year ago

  • Priority changed from Urgent to Immediate

wrongly changed priority of ticket

#34

Updated by Ram Kordale over 1 year ago

  • Status changed from Feedback to Closed

Also available in: Atom PDF