@clickview/ship-it 0.0.16 → 0.0.18
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/A2/A2 Testing Notes.csv +518 -0
- package/dist/A2/Batch.Api.md +37 -0
- package/dist/A2/ClickView.Curator.md +17 -0
- package/dist/A2/Clients.md +23 -0
- package/dist/A2/Discover.Api.md +14 -0
- package/dist/A2/FileGenerator.md +8 -0
- package/dist/A2/Krakend.md +25 -0
- package/dist/A2/Library.Api.md +13 -0
- package/dist/A2/Lite.md +27 -0
- package/dist/A2/Master.Auth.md +52 -0
- package/dist/A2/Objects.Api.md +8 -0
- package/dist/A2/Permissions.Api.md +12 -0
- package/dist/A2/Release Notes.md +479 -0
- package/dist/A2/Search.Api.md +12 -0
- package/dist/A2/Subjects.API.md +22 -0
- package/dist/A2/Users.Api.md +63 -0
- package/dist/A2/clickview.md +83 -0
- package/dist/A2/index.app.md +15 -0
- package/dist/Z2/Analytics Web.md +7 -0
- package/dist/Z2/Batch.Api.md +13 -0
- package/dist/Z2/ClickView Online.md +19 -0
- package/dist/Z2/ClickView.Cron.md +8 -0
- package/dist/Z2/ClickView.Curator.md +39 -0
- package/dist/Z2/Clients.md +31 -0
- package/dist/Z2/Discover.Api.md +12 -0
- package/dist/Z2/Domain.Api.md +31 -0
- package/dist/Z2/Image.Api.md +12 -0
- package/dist/Z2/Krakend.md +33 -0
- package/dist/Z2/LTI.md +8 -0
- package/dist/Z2/Launch.Web.md +9 -0
- package/dist/Z2/Library Editor.md +7 -0
- package/dist/Z2/Library.Api.md +13 -0
- package/dist/Z2/Lite.md +32 -0
- package/dist/Z2/Master.Auth.md +16 -0
- package/dist/Z2/Objects.Api.md +9 -0
- package/dist/Z2/Prospect.Api.md +17 -0
- package/dist/Z2/RealtimeSalesforceSync.md +19 -0
- package/dist/Z2/Release Notes.md +549 -0
- package/dist/Z2/Users.Api.md +13 -0
- package/dist/Z2/clickview.md +135 -0
- package/lib/src/constants/MonorepoPackageJsonFiles.js +2 -1
- package/lib/src/constants/ProjectNames.js +1 -0
- package/lib/src/constants/TeamCityBuildIDs.js +1 -0
- package/lib/src/tasks/InstallPackagesTask.js +3 -1
- package/lib/src/tasks/ValidateTask.js +3 -2
- package/package.json +1 -1
- package/dist/A3/ClickView Online.md +0 -28
- package/dist/A3/ClickView.Curator.md +0 -19
- package/dist/A3/Clients.md +0 -21
- package/dist/A3/Domain.Api.md +0 -15
- package/dist/A3/Krakend.md +0 -33
- package/dist/A3/Library Editor.md +0 -0
- package/dist/A3/Lite.md +0 -35
- package/dist/A3/Master.Api.md +0 -10
- package/dist/A3/Master.Auth.md +0 -51
- package/dist/A3/Release Notes.md +0 -683
- package/dist/A3/Subjects.API.md +0 -18
- package/dist/A3/Users.Api.md +0 -89
- package/dist/A3/clickview.md +0 -325
- package/dist/X3/X3 Testing Notes.csv +0 -1222
- package/dist/X3/X3 Testing Notes.xlsx +0 -0
- package/dist/Z1/Analytics Web.md +0 -0
- package/dist/Z1/ClickView Online.md +0 -24
- package/dist/Z1/ClickView.Cron.md +0 -8
- package/dist/Z1/ClickView.Curator.md +0 -28
- package/dist/Z1/ClickView.EmailService.md +0 -17
- package/dist/Z1/Clients.md +0 -29
- package/dist/Z1/Discover.Api.md +0 -20
- package/dist/Z1/Domain.Api.md +0 -34
- package/dist/Z1/FileGenerator.md +0 -13
- package/dist/Z1/Image.Api.md +0 -14
- package/dist/Z1/Krakend.md +0 -8
- package/dist/Z1/Library Editor.md +0 -0
- package/dist/Z1/Lite.md +0 -47
- package/dist/Z1/Master.Auth.md +0 -43
- package/dist/Z1/Objects.Api.md +0 -30
- package/dist/Z1/Permissions.Api.md +0 -10
- package/dist/Z1/Prospect.Api.md +0 -16
- package/dist/Z1/RealtimeSalesforceSync.md +0 -16
- package/dist/Z1/Release Notes.md +0 -855
- package/dist/Z1/Search.Api.md +0 -19
- package/dist/Z1/Share.Api.md +0 -12
- package/dist/Z1/Ship It.md +0 -23
- package/dist/Z1/Subjects.API.md +0 -16
- package/dist/Z1/Users.Api.md +0 -26
- package/dist/Z1/VideoProcessorService.md +0 -11
- package/dist/Z1/Z1 Testing Notes.csv +0 -1079
- package/dist/Z1/Z1 Testing Notes.xlsx +0 -0
- package/dist/Z1/clickview.md +0 -316
- /package/dist/{A3/Analytics Web.md → Z2/Permissions.Api.md} +0 -0
|
@@ -1,1079 +0,0 @@
|
|
|
1
|
-
projectName,mrTitle,mrAuthor,mrAssignee,mrUrl,notes
|
|
2
|
-
clickview,Curation studio - Fix audience in playlist widget,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2974,"## Curator
|
|
3
|
-
|
|
4
|
-
### Code changes
|
|
5
|
-
- Update playlist request key to correctly fetch playlist with `subjectPresentationAudience`. Fix INBOX-4414
|
|
6
|
-
- Update playlist request key to correctly fetch playlist with `subjectPresentationAudience`. Fix INBOX-4426
|
|
7
|
-
|
|
8
|
-
## Testing
|
|
9
|
-
- In the curation studio, adding a playlist widget, when adding playlists, the audience information should be displayed correctly."
|
|
10
|
-
clickview,Add ship it config,Cameron Hill,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2973,"## Monorepo
|
|
11
|
-
|
|
12
|
-
### Code changes
|
|
13
|
-
- Add `ship-it.config` file for latest changes in ship it https://gitlab.cvinternal.net/clickview/front-end/ship-it/-/merge_requests/42
|
|
14
|
-
|
|
15
|
-
## Testing
|
|
16
|
-
- Ship it runs"
|
|
17
|
-
clickview,Lite: autofocus search bar on mobile,Joshua Taylor,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2972,"## Lite
|
|
18
|
-
|
|
19
|
-
### Customer changes
|
|
20
|
-
- On mobile, the search bar will now automatically focus when opened up.
|
|
21
|
-
|
|
22
|
-
## Testing
|
|
23
|
-
- In NP on mobile, the search bar should now automatically focus when opened up. On desktop, it should not automatically focus ever."
|
|
24
|
-
clickview,Lite: Parent video,Shale Kuzmanovski,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2971,"## Dependencies
|
|
25
|
-
https://gitlab.cvinternal.net/clickview/back-end/discover.api/-/merge_requests/103
|
|
26
|
-
|
|
27
|
-
## Shared
|
|
28
|
-
|
|
29
|
-
### Code changes
|
|
30
|
-
- Now fetching the parent video for clips and interactives using the `_parentVideo` virtual assoc.
|
|
31
|
-
- Now fetching the parent clip for interactives using `_parentClip`.
|
|
32
|
-
- Updated our types for clips and interactives to no longer have `video` or `clip` properties, instead they have `_parentVideo` or `_parentClip` properties, whose type is a subset of the video and clip types.
|
|
33
|
-
|
|
34
|
-
## Testing
|
|
35
|
-
- The way clips and interactives fetch their parent video has been reworked in order to accommodate a share change, which means clips and interactives will need a regression test. Testing that the video item in lists or widgets, as well as their pages behave properly.
|
|
36
|
-
- Playlists
|
|
37
|
-
- Watch History
|
|
38
|
-
- My Videos
|
|
39
|
-
- Favourite videos
|
|
40
|
-
- Student feed
|
|
41
|
-
- Student feed in a playlist
|
|
42
|
-
- Beneath the video on the page page
|
|
43
|
-
- Search results
|
|
44
|
-
- Share links, both full page and embed
|
|
45
|
-
- Creating clips
|
|
46
|
-
- Creating interactives on clips
|
|
47
|
-
- In analytics, test that viewing interactive results, and the top videos feature work."
|
|
48
|
-
clickview,Curator: Fix missing audience,Joshua Taylor,Nan Chen,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2970,"## Curator
|
|
49
|
-
|
|
50
|
-
### Customer changes
|
|
51
|
-
- Fixed an issue in curation studio where if you were managing a dashboard or topic in a presentation that isn't associated with your account, the audiences would always display as ""Missing Audience"" even if it had one. Fixes CC-471
|
|
52
|
-
|
|
53
|
-
## Testing"
|
|
54
|
-
clickview,Public: Curriculum links,Michael McDonnell,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2968,"## Before release
|
|
55
|
-
|
|
56
|
-
### other
|
|
57
|
-
- [ ] Curriculums used in the public product need to be added to `public_presentations` table in Subject API
|
|
58
|
-
- [ ] After releasing, we need to update the `curriculums.view` permission in production (this is technically an `After release` action. It **must** happen **after** pushing, otherwise we will break public play pages and send unauthenticated users to the sign in page)
|
|
59
|
-
- `PUT v1/permissions/35` the following payload (please sanity check the id before updating - the permission already existing in both staging and production):
|
|
60
|
-
- The change below is to add `public` to the `permissionSet` property. This will allow public users to be able to view curriculum links.
|
|
61
|
-
```
|
|
62
|
-
{
|
|
63
|
-
""id"": 35,
|
|
64
|
-
""name"": ""curriculums.view"",
|
|
65
|
-
""description"": ""Can view curriculum links"",
|
|
66
|
-
""visibility"": ""Internal"",
|
|
67
|
-
""type"": ""Static"",
|
|
68
|
-
""dateCreated"": ""2023-02-14T03:26:39Z"",
|
|
69
|
-
""permissionSets"": [
|
|
70
|
-
""public""
|
|
71
|
-
],
|
|
72
|
-
""roles"": [
|
|
73
|
-
""Admin"",
|
|
74
|
-
""SuperAdmin""
|
|
75
|
-
],
|
|
76
|
-
""userTypes"": [
|
|
77
|
-
""Staff""
|
|
78
|
-
]
|
|
79
|
-
}
|
|
80
|
-
```
|
|
81
|
-
|
|
82
|
-
## Dependencies
|
|
83
|
-
- https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/348
|
|
84
|
-
- https://gitlab.cvinternal.net/clickview/back-end/clickview.subjects.api/-/merge_requests/189
|
|
85
|
-
|
|
86
|
-
## Lite
|
|
87
|
-
|
|
88
|
-
### Customer changes
|
|
89
|
-
- Curriculum links are now accessible in the public version of NP
|
|
90
|
-
- Implements LTE-1759
|
|
91
|
-
- Fixed an error where changing presentations as a public user could send them to the login page
|
|
92
|
-
- Fixes INBOX-4490
|
|
93
|
-
- Switching presentation will now be a little faster than usual
|
|
94
|
-
|
|
95
|
-
### Code changes
|
|
96
|
-
- Refactored `useTopicPresentation` to become `usePresentation` that now supports use of both topic and curriculum presentations
|
|
97
|
-
- Cleaned up `useCurriculums` to utilize `usePresentation`
|
|
98
|
-
- Updated play page components to consistently use curriculums
|
|
99
|
-
- Update curriculum handling in search results
|
|
100
|
-
|
|
101
|
-
## Shared
|
|
102
|
-
|
|
103
|
-
### Code changes
|
|
104
|
-
- Update public support for curriculum requests
|
|
105
|
-
- Clean up `publicUrl` for presentation requests
|
|
106
|
-
- Gracefully handle missing curriculum in `CurriculumDropdown`
|
|
107
|
-
- Update invalidation keys when updating presentation
|
|
108
|
-
|
|
109
|
-
## Testing
|
|
110
|
-
- In NP:
|
|
111
|
-
- As an authenticated user:
|
|
112
|
-
- Accessing curriculum links on the play page as a staff/admin should continue to load as expected (regression)
|
|
113
|
-
- Curriculum links should load (for accounts with one or many available curriculums)
|
|
114
|
-
- Switching curriculums should work as expected and correctly load the links for the nominated curriculum
|
|
115
|
-
- It should be possible to access the curriculum link applink from both V@G and video details page
|
|
116
|
-
- Accessing curriculum links on a video list view as a staff/admin should continue to load as expected (regression)
|
|
117
|
-
- Curriculum links should load (for accounts with one or many available curriculums)
|
|
118
|
-
- Switching curriculums should work as expected and correctly load the links for the nominated curriculum
|
|
119
|
-
- Accessing curriculum links in search results as a staff/admin should continue to load as expected (regression)
|
|
120
|
-
- Curriculum links shouldn't be accessible to students
|
|
121
|
-
- Page loading should loading the user's last nominated presentation (when switching between many)
|
|
122
|
-
- Videos access via share link should load as expected and the curriculums should be viewable
|
|
123
|
-
- As a public/unauthenticated user:
|
|
124
|
-
- It should be possible to view curriculum links on the play page
|
|
125
|
-
- Curriculum links should load (AU and US will have many curriculums available. GB and GB-SCT will only have access to one curriculum each).
|
|
126
|
-
- Switching curriculums should work as expected and correctly load the links for the nominated curriculum
|
|
127
|
-
- It should be possible to access the curriculum link applink from both V@G and video details page
|
|
128
|
-
- We currently don't show any actions on public video list views. For the time being this will remain.
|
|
129
|
-
- Search results, video lists in both series and playlists should continue to load as expected (regression)
|
|
130
|
-
- Nominated curriculums should be persisted in local storage in the same way that we do for topic presentations
|
|
131
|
-
- Switching presentations should no longer show an error in the console
|
|
132
|
-
- Switching presentations should no longer send the user to the login page if they reload after switching"
|
|
133
|
-
clickview,Styles: Fix tooltip styling,Shale Kuzmanovski,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2967,"## Styles
|
|
134
|
-
|
|
135
|
-
### Customer changes
|
|
136
|
-
- Fix the styling of our tooltip, it no longer will show the border bottom across the arrow.
|
|
137
|
-
|
|
138
|
-
## Testing
|
|
139
|
-
- Hover over a video you do not have access to, take a deep breath of relaxation because that god forsaken border styling issue is gone."
|
|
140
|
-
clickview,Add link resources to what's included tag,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2966,"## Shared
|
|
141
|
-
|
|
142
|
-
### Code changes
|
|
143
|
-
- Fix ""What's included"" doesn't include link resources. Fix LTE-1754
|
|
144
|
-
|
|
145
|
-
## Testing
|
|
146
|
-
- In the classifìcation dashboard (Lite and Curator), the 'what's included' tag of a video, the resources section should also contain the link resources of the current video."
|
|
147
|
-
clickview,Curator: Prevent editing published dashboards,Joshua Taylor,Nan Chen,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2965,"## Curator
|
|
148
|
-
|
|
149
|
-
### Customer changes
|
|
150
|
-
- In curation studio, published dashboards can no longer be edited. Curators now have to create a copy of the published dashboard, then make changes to the copy, then publish that. Implements CC-464
|
|
151
|
-
|
|
152
|
-
## Testing
|
|
153
|
-
- In curation studio, you should no longer be allowed to edit a published dashboard.
|
|
154
|
-
- In curation studio, you should still be able to edit draft dashboards."
|
|
155
|
-
clickview,Common: Fix import of lodash.cloneDeep,Shale Kuzmanovski,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2963,"## Common
|
|
156
|
-
|
|
157
|
-
### Code changes
|
|
158
|
-
- Importing cloneDeep using `lodash.clonedeep` to ensure the rest of lodash isn't coming with it.
|
|
159
|
-
|
|
160
|
-
## Monorepo
|
|
161
|
-
|
|
162
|
-
### Code changes
|
|
163
|
-
- Adding a type for cloneDeep to all projects.
|
|
164
|
-
|
|
165
|
-
## Testing
|
|
166
|
-
- Every frontend project needs a regression test, clicking around to a few pages (Online, Curator, Lite, Reports)"
|
|
167
|
-
clickview,Curator: Copy dashboard,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2962,"## Dependencies
|
|
168
|
-
- https://gitlab.cvinternal.net/clickview/back-end/discover.api/-/merge_requests/102
|
|
169
|
-
|
|
170
|
-
## Curator
|
|
171
|
-
|
|
172
|
-
### Customer changes
|
|
173
|
-
- Curators can now create a copy of discover and topic dashboards. Implements CC-465
|
|
174
|
-
|
|
175
|
-
## Testing
|
|
176
|
-
- In Curator, you should now be able to copy discover and topic dashboards using the ""Copy dashboard"" action on the dashboards list page. Editing a copy should not affect the original dashboard."
|
|
177
|
-
clickview,Lite: Student sign up,Shale Kuzmanovski,Nischal Shrestha,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2959,"## Lite
|
|
178
|
-
|
|
179
|
-
### Customer changes
|
|
180
|
-
- During the sign up workflow, students are now prompted to enter their date of birth before we collect their email address. We do not collect and PII from students who are under the age of 13, instead they are sent to our free videos on the marketing website. Once a student has filled out our date of birth form, they cannot get back to the sign up form, it will always show them the link to the free videos page. This is done in order to protect young students who cannot consent to their data being collection, in compliance with COPPA in the United States of America.
|
|
181
|
-
|
|
182
|
-
### Code changes
|
|
183
|
-
- Re-worked our completion step to take a `CompletionStepId`. We now have 3 completion step types, trial status, added to braze, and young student.
|
|
184
|
-
|
|
185
|
-
## Testing
|
|
186
|
-
- Sign up for a trial as a student, enter date of birth that is younger than 13 years of age. Ensure you are linked to the marketing free videos page. Ensure that if you go back to the sign up form you are always sent to the free videos page."
|
|
187
|
-
clickview,Bootstrap permissions and products,Cameron Hill,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2956,"## Dependencies
|
|
188
|
-
- https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/346
|
|
189
|
-
|
|
190
|
-
## Lite
|
|
191
|
-
|
|
192
|
-
### Code changes
|
|
193
|
-
- Add bootstrap keys for `permissions` and `products`
|
|
194
|
-
|
|
195
|
-
## Testing
|
|
196
|
-
- On the initial page load, permissions and products should not be fetched
|
|
197
|
-
- After the flight cache expires, permissions and products should be fetched correctly"
|
|
198
|
-
clickview,Remove forcedMediaQuery prop,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2955,"## Shared
|
|
199
|
-
|
|
200
|
-
### Code changes
|
|
201
|
-
- Removed the `forcedMediaQuery` prop that's unused and littered throughout the codebase.
|
|
202
|
-
|
|
203
|
-
## Testing"
|
|
204
|
-
clickview,Make alert smaller in mobile,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2954,"## Shared
|
|
205
|
-
|
|
206
|
-
### Customer changes
|
|
207
|
-
- Alert in mobile will display 'x' icon at the first line.
|
|
208
|
-
|
|
209
|
-
### Code changes
|
|
210
|
-
- Shrink the width of alert in mobile. Fixes INBOX-3963.
|
|
211
|
-
|
|
212
|
-
## Testing
|
|
213
|
-
- In NP, in small size(320px - 575px), the close icon in alert will be at the first line. Can test with 'favourited' any video."
|
|
214
|
-
clickview,Tooling: I18n Watcher,Shale Kuzmanovski,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2952,"## Dependencies
|
|
215
|
-
- https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/345
|
|
216
|
-
|
|
217
|
-
## Tooling
|
|
218
|
-
|
|
219
|
-
### Code changes
|
|
220
|
-
- The `I18nPlugin` will now watch all the `*.lang.json` files for file changes, and output the updated language file.
|
|
221
|
-
|
|
222
|
-
## Testing"
|
|
223
|
-
clickview,User Manager: Fix downloading CSV files in safari,Joshua Taylor,Nan Chen,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2951,"## Curator
|
|
224
|
-
|
|
225
|
-
### Customer changes
|
|
226
|
-
- Fixed an issue in user manager where the bulk upload template file couldn't be downloaded in Safari. Fixes INBOX-4434
|
|
227
|
-
|
|
228
|
-
## Lite
|
|
229
|
-
|
|
230
|
-
### Customer changes
|
|
231
|
-
- Fixed an issue in user manager where the bulk upload template file couldn't be downloaded in Safari. Fixes INBOX-4434
|
|
232
|
-
|
|
233
|
-
## Online
|
|
234
|
-
|
|
235
|
-
### Customer changes
|
|
236
|
-
- Fixed an issue in user manager where the bulk upload template file couldn't be downloaded in Safari. Fixes INBOX-4434
|
|
237
|
-
|
|
238
|
-
## Testing
|
|
239
|
-
- Open user manager in safari in either NP, Curator, or Online. Go to bulk upload, and click to download the template. The file should download."
|
|
240
|
-
clickview,Lite: Upload button text wrapping,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2950,"## Testing
|
|
241
|
-
- When uploading a video in NP, the ""Upload video"" text in the button shouldn't wrap at any screen size. Fixes INBOX-4436"
|
|
242
|
-
clickview,Library Editor: Fix thumbnail instant search results,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2949,"## Library Editor
|
|
243
|
-
|
|
244
|
-
### Customer changes
|
|
245
|
-
- Fixed an issue where video results in the search dropdown displayed very small thumbnails. Fixes INBOX-4457
|
|
246
|
-
|
|
247
|
-
## Testing
|
|
248
|
-
- In library editor's search bar dropdown, video result thumbnails should look normal. They shouldn't look like how they do in the screenshot on this ticket: https://clickview.atlassian.net/browse/INBOX-4457"
|
|
249
|
-
clickview,Lite: Explore all subjects ordering,Joshua Taylor,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2948,"## Lite
|
|
250
|
-
|
|
251
|
-
### Customer changes
|
|
252
|
-
- The ""explore all subjects"" page should now always display the subjects in alphabetical order. Fixes INBOX-4048
|
|
253
|
-
|
|
254
|
-
## Testing
|
|
255
|
-
- Subjects on the ""Explore all subjects"" page should be in alphabetical order"
|
|
256
|
-
clickview,User manager - select user with spreadsheet,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2943,"## Dependencies
|
|
257
|
-
- https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/343
|
|
258
|
-
- https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1056
|
|
259
|
-
- https://gitlab.cvinternal.net/clickview/front-end/ClickView.Online/-/merge_requests/3194
|
|
260
|
-
|
|
261
|
-
## Curator
|
|
262
|
-
|
|
263
|
-
### Customer changes
|
|
264
|
-
- In the user manager page, users can now bulk-select by uploading a spreadsheet containing the student/staff they want to select. Implements LTE-1726
|
|
265
|
-
|
|
266
|
-
## Lite
|
|
267
|
-
|
|
268
|
-
### Customer changes
|
|
269
|
-
- In Lite user manager page, users can now bulk-select by uploading a spreadsheet containing the student/staff they want to select. Implements LTE-1726
|
|
270
|
-
|
|
271
|
-
## Online
|
|
272
|
-
|
|
273
|
-
### Customer changes
|
|
274
|
-
- In the user manager page, users can now bulk-select by uploading a spreadsheet containing the student/staff they want to select. Implements LTE-1726
|
|
275
|
-
|
|
276
|
-
## Shared
|
|
277
|
-
|
|
278
|
-
### Code changes
|
|
279
|
-
- Add SelectWithSpreadSheet component
|
|
280
|
-
- Add a helper function to build & split query params to fetch user
|
|
281
|
-
- Fixed an error during parsing an empty CSV file
|
|
282
|
-
|
|
283
|
-
## Testing
|
|
284
|
-
- In Lite/Curator/Online, the user manager page for students and staff, the 'Select with spreadsheet' action was added to the actions dropdown. It should find and select all the valid users from the spreadsheet in the current schools.
|
|
285
|
-
- It should find both verified and unverified users.
|
|
286
|
-
- It should not find users from other schools.
|
|
287
|
-
- Uploading an empty CSV file should trigger the error popup.
|
|
288
|
-
- CSV doesn't contain any of the Email and Username columns will trigger the error popup.
|
|
289
|
-
- It should find users with the correct user type, e.g., in the student manager page, it will only find all the students in the spreadsheet.
|
|
290
|
-
- Notes to Jav: Online requires more attention if you are able to do a deep test.
|
|
291
|
-
- Regression test: in Lite/Curator/Online, the user manager page for students and staff, click on the 'Add new student/staff', and choose to upload a CSV that only contains headings but no content should trigger the correct warning popup."
|
|
292
|
-
clickview,Draft: User manager - select with spreadsheet,Nan Chen,,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2942,
|
|
293
|
-
clickview,Topics mobile,Joshua Taylor,Nan Chen,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2941,"## Lite
|
|
294
|
-
|
|
295
|
-
### Customer changes
|
|
296
|
-
- When loading into a specific table of contents section on a topic page, you should land a bit closer than before. Implements LTE-1775 and fixes INBOX-4453
|
|
297
|
-
- Fixed an issue on mobile where the left nav would initially be expanded then immediately collapsed, resulting in a harsh first landing experience. Implements LTE-1775
|
|
298
|
-
- Fixed the partial loading for the breadcrumbs, audience filter, and banner on the topic page not matching up with the actual UI accurately Implements LTE-1775
|
|
299
|
-
- Adjusted the partial loading for 4 video widgets (Implements LTE-1775):
|
|
300
|
-
- ""Enhanced large slider""
|
|
301
|
-
- ""Featured video""
|
|
302
|
-
- ""Hero video""
|
|
303
|
-
- ""2 side by side thumbnails""
|
|
304
|
-
- Fixed the alignment of the thumbnail and video title text on the featured media widget on smaller screens Implements LTE-1775
|
|
305
|
-
|
|
306
|
-
## Testing
|
|
307
|
-
- The partial loading on the topics page should look really really accurate on:
|
|
308
|
-
- Mobile
|
|
309
|
-
- IPad
|
|
310
|
-
- Desktop
|
|
311
|
-
- Curated topics pages on mobile should not have any UI issues."
|
|
312
|
-
clickview,Lite: Prevent student feedback,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2934,"## Before release
|
|
313
|
-
|
|
314
|
-
### other
|
|
315
|
-
- Create a `feedback.submit` permission in permission API for staff and admins only. Not students.
|
|
316
|
-
|
|
317
|
-
## Lite
|
|
318
|
-
|
|
319
|
-
### Customer changes
|
|
320
|
-
- Fixed an issue where students could provide feedback via the footer on the discover page.
|
|
321
|
-
|
|
322
|
-
## Testing
|
|
323
|
-
- As a student, you should no longer see an option to provide feedback via the form at the bottom of the discover page."
|
|
324
|
-
clickview,Lite: Remove smooth scrolling,Joshua Taylor,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2933,"## Lite
|
|
325
|
-
|
|
326
|
-
### Customer changes
|
|
327
|
-
- Fixed an issue where all pages had smooth scrolling. Fixes LTE-1762
|
|
328
|
-
|
|
329
|
-
## Testing
|
|
330
|
-
- When navigating between 2 pages, it should not ""smooth scroll"" between them.
|
|
331
|
-
- On a curated topics page with a able of contents, the TOC should still have smooth scrolling behavior."
|
|
332
|
-
clickview,Curation Studio: Search within presentation,Joshua Taylor,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2932,"## Curator
|
|
333
|
-
|
|
334
|
-
### Customer changes
|
|
335
|
-
- When curating a topic page, curators can now search within the entire presentation rather than just within the topic. Implements CC-466
|
|
336
|
-
|
|
337
|
-
## Testing
|
|
338
|
-
- In Curation Studio when editing a topic page, you should now see an option to search within the whole presentation when creating a widget."
|
|
339
|
-
clickview,Lite - discover dashboard widget video-list view,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2929,"## Lite
|
|
340
|
-
|
|
341
|
-
### Customer changes
|
|
342
|
-
- In the lite discover dashboard, for three types of widget templates (1. Enhanced large slider 2. Large slider 3. Small slider), their title can be clicked and navigate to a video list view of that widget. Implements LTE-1727.
|
|
343
|
-
|
|
344
|
-
### Code changes
|
|
345
|
-
- Add a new CuratedDashbaordWidgetView based on the domain option.
|
|
346
|
-
- Add new appLinks type to the dashboard.
|
|
347
|
-
|
|
348
|
-
## Testing
|
|
349
|
-
- In the lite discover dashboard, for three types of widget templates (1. Enhanced large slider 2. Large slider 3. Small slider), their title can be clicked and navigate to a video list view of that widget containing all the videos within that widget."
|
|
350
|
-
clickview,Curated topics - close video guidance on tap,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2927,"## Lite
|
|
351
|
-
|
|
352
|
-
### Code changes
|
|
353
|
-
- Add event listener to video guidance tag to set the active status of popout. Fix LTE-1728.
|
|
354
|
-
|
|
355
|
-
## Testing
|
|
356
|
-
- In lite, view the classification dashboard on a mobile web screen. The video guidance of the `Hero video` widget and `2 side by side` widget can be opened and closed by tapping the badge."
|
|
357
|
-
clickview,Fix student video activity badge size,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2926,"## Lite
|
|
358
|
-
|
|
359
|
-
### Code changes
|
|
360
|
-
- Shrink the activity status badge size. Fix LTE-1744
|
|
361
|
-
|
|
362
|
-
## Testing"
|
|
363
|
-
clickview,Signup: Add to braze step item,Nischal Shrestha,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2924,"## Lite
|
|
364
|
-
|
|
365
|
-
### Customer changes
|
|
366
|
-
- If a prospects school is not supported they now can request a trial like in marketing. Implements LTE-1733.
|
|
367
|
-
|
|
368
|
-
### Code changes
|
|
369
|
-
- Added a new sign up step named `add-to-braze`. Implements LTE-1752.
|
|
370
|
-
- Added a form that add a prospect with all their details to braze. Implements LTE-1763.
|
|
371
|
-
- Updated terms and condition link to point to global marketing website. Fixes LTE-1735.
|
|
372
|
-
|
|
373
|
-
## Testing
|
|
374
|
-
- In Lite, start the sign up process and select a school which has a trial status of 'unavailable'
|
|
375
|
-
- Click get started and we should be taken to 'add-to-braze' step which asks more information about the user
|
|
376
|
-
- The form is validated client side and should not allow for
|
|
377
|
-
- strings larger than 255 characters
|
|
378
|
-
- strings starting with `$` (braze thing)
|
|
379
|
-
- phone number can only be digits between 4 and 255 characters long
|
|
380
|
-
- Clicking submit button should send the user data to braze and we should be able to look up the user with that email in it.
|
|
381
|
-
- The user should also have a custom event named `marketing_free_trial_signup`"
|
|
382
|
-
clickview,Common: deepCopy() fixes,Shale Kuzmanovski,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2921,"## Common
|
|
383
|
-
|
|
384
|
-
### Code changes
|
|
385
|
-
- Updated the recursive denormalizer test to cater for a use case that wasn't covered.
|
|
386
|
-
- Updated `ObjectHelper.deepClone()` to use `cloneDeep` from lodash. Addresses INBOX-4054.
|
|
387
|
-
|
|
388
|
-
## Lite
|
|
389
|
-
|
|
390
|
-
### Code changes
|
|
391
|
-
- Fixed a bug that was causing us to drop events when logging our playlists.
|
|
392
|
-
- Fixed a but that was causing the edit video details to break after favouriting the video. Fixes INBOX-4292.
|
|
393
|
-
|
|
394
|
-
## Shared
|
|
395
|
-
|
|
396
|
-
### Code changes
|
|
397
|
-
- Cleaned up code in `EditVideoDetails` to only pass the relevant form data as initial values.
|
|
398
|
-
- Cleaned up the `<FormWrapper />`, it will no longer recursively use `ObjectHelper.deepClone()`
|
|
399
|
-
- Updated the `ReorderHelper` to pass only the object ids back into flight. Fixes STG-271
|
|
400
|
-
|
|
401
|
-
## Testing
|
|
402
|
-
- Playlist re-ordering needs to be tested in Lite and NP. In Lite, it needs to be tested on a video that has clips and interactives, after loading to the play page and then going back to the playlist to re-order. Previously this threw an error https://clickview.atlassian.net/browse/STG-271
|
|
403
|
-
- Favourite a video in my videos and then make sure the edit details page loads.
|
|
404
|
-
- Because one of our core utilities were replaced, there are quite a few places that need a very brief regression test. We're just looking for the UI behaving normally
|
|
405
|
-
- Direct sharing to classrooms in the NP share popup
|
|
406
|
-
- The playlist play page in online
|
|
407
|
-
- The audience filter in reports
|
|
408
|
-
- Editing a workspace video in Online and NP
|
|
409
|
-
- Bulk uploading users to user manager (Just NP is enough)
|
|
410
|
-
- Approving and rejecting library requests in library editor
|
|
411
|
-
- Editing a clip's details in NP, make sure changing the name of the clip doesn't modify the start and end time"
|
|
412
|
-
clickview,Fix table scroll button of interactive results,Nan Chen,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2918,"## Analytics
|
|
413
|
-
|
|
414
|
-
### Code changes
|
|
415
|
-
- Fix the missing scroll button in the results table. Fix INBOX-4407.
|
|
416
|
-
|
|
417
|
-
## Testing
|
|
418
|
-
- In analytics, checking the results of an interactive, when there are more than 13 questions, the scroll to right button should be displayed."
|
|
419
|
-
clickview,Lite: Fix sign in button not being an anchor,Shale Kuzmanovski,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2917,"## Lite
|
|
420
|
-
|
|
421
|
-
### Code changes
|
|
422
|
-
- Fixed the sign in button on the public play pages not being an anchor. Fixes INBOX-4372
|
|
423
|
-
|
|
424
|
-
## Testing"
|
|
425
|
-
clickview,Add spell check to annotation editor,Nischal Shrestha,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2916,"## Player
|
|
426
|
-
|
|
427
|
-
### Customer changes
|
|
428
|
-
- Enable spellcheck for annotation editor when editing interactives. Fixes INBOX-4357.
|
|
429
|
-
|
|
430
|
-
### Code changes
|
|
431
|
-
- Added `spellCheck` for DraftJs editor since it render a `div` with `contentEditable=true`
|
|
432
|
-
|
|
433
|
-
## Testing
|
|
434
|
-
- In chrome settings, make sure to enable spellcheck. visit `chrome://settings/?search=spell+check`
|
|
435
|
-
- In Lite, create a new interactive and add an annotation.
|
|
436
|
-
- If we start typing in the editor and misspell any words we should trigger the spellcheck."
|
|
437
|
-
clickview,Lite: Fix topic filter disappearing,Joshua Taylor,Nan Chen,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2915,"## Lite
|
|
438
|
-
|
|
439
|
-
### Customer changes
|
|
440
|
-
- Fixed an issue on topics where if you apply a filter that reveals an empty state, then you click ""Clear"", the page doesn't revert back to the filterless view and the audience filters disappear. Fixes INBOX-4412
|
|
441
|
-
|
|
442
|
-
## Testing
|
|
443
|
-
- On a topics page, apply a filter so that all content disappears. This should reveal an empty state. Then, click ""Clear"" to clear the filters. The page should revert back to the original filterless view."
|
|
444
|
-
clickview,Lite: Fix share popup showing display name for teachers,Joshua Taylor,Nischal Shrestha,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2913,"## Lite
|
|
445
|
-
|
|
446
|
-
### Customer changes
|
|
447
|
-
- Fixed a bug where if a teacher is direct sharing to another teacher, the search dropdown would display the teacher's display name. Fixes INBOX-4362
|
|
448
|
-
- Fixed a but where if a teacher is adding another teacher to a classroom, the search dropdown would display the teacher's display name. Fixes INBOX-4362
|
|
449
|
-
|
|
450
|
-
## Testing
|
|
451
|
-
- In NP, when sharing to another teacher, they should not appear with their `displayName` but rather their full name.
|
|
452
|
-
- In NP, when adding another teacher to a classroom, they should not appear in the search dropdown with their `displayName` but rather their full name."
|
|
453
|
-
clickview,Signup fixes,Nischal Shrestha,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2911,"## Lite
|
|
454
|
-
|
|
455
|
-
### Code changes
|
|
456
|
-
- Updated usage of hardcoded country codes to use `LocalityHelper`.
|
|
457
|
-
- 'Use my current location' is selectable when navigating by keyboard.
|
|
458
|
-
- 'Can't find your school?' is selectable when navigating by keyboard.
|
|
459
|
-
- Fixed a bug where clearing SearchLocalityInput in Cant find school step clearing the school name.
|
|
460
|
-
- Fixed style imports to be relative.
|
|
461
|
-
|
|
462
|
-
## Testing
|
|
463
|
-
- In Signup App, Go through the steps until you reach searching school step.
|
|
464
|
-
- Search for a locality and we should be able to navigate to 'Use my location' then press enter
|
|
465
|
-
- Seach for a school and we should be able to navigate to 'Cant find your school?' then press enter
|
|
466
|
-
- We should be able to submit the form as normal and get a free trial."
|
|
467
|
-
clickview,Clean up - Topic banner in web mobile,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2910,"## Lite
|
|
468
|
-
|
|
469
|
-
### Code changes
|
|
470
|
-
- Fixed banner height in topic classification board at web-mobile screen size. Fix INBOX-4410
|
|
471
|
-
- Hide the text of follow/unfollow button at the web-mobile screen size. Fix INBOX-4410
|
|
472
|
-
|
|
473
|
-
## Testing"
|
|
474
|
-
clickview,Clean up - Update overlay style on small video thumbnail,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2906,"## Shared
|
|
475
|
-
|
|
476
|
-
### Code changes
|
|
477
|
-
- Shrink the padding of the overlay title of the video thumbnail on the web-mobile screen. Fix INBOX-4385
|
|
478
|
-
|
|
479
|
-
## Testing"
|
|
480
|
-
clickview,Clean up - Update editing layout for editable subject web-mobile,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2905,"## Shared
|
|
481
|
-
|
|
482
|
-
### Code changes
|
|
483
|
-
- Update the layout of the editing status for `EditableCoverFixedPillListSmallWidgetTemplate`. Implements INBOX-4384
|
|
484
|
-
|
|
485
|
-
## Testing"
|
|
486
|
-
clickview,Lite: Sign up UI updates,Shale Kuzmanovski,Nischal Shrestha,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2904,"## Lite
|
|
487
|
-
|
|
488
|
-
### Customer changes
|
|
489
|
-
- Styling of the sign up form has been slightly updated to be more in line with our Auth forms.
|
|
490
|
-
- The email, location and school inputs in the sign up for now have clear buttons to clear their content.
|
|
491
|
-
- The location and school input typeaheads now have keyboard support: Down, up arrows as well as enter will now work as expected.
|
|
492
|
-
|
|
493
|
-
### Code changes
|
|
494
|
-
- Cleaning up markup and utility classes across all steps of the sign up app.
|
|
495
|
-
|
|
496
|
-
## Testing
|
|
497
|
-
- Regression test signing up."
|
|
498
|
-
clickview,Clean up - audiences & breadcrumbs in classification dashboard in web-mobile,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2903,"## Lite
|
|
499
|
-
|
|
500
|
-
### Code changes
|
|
501
|
-
- Only show the level above for breadcrumb in web-mobile. Fix INBOX-4382;
|
|
502
|
-
- Increase the padding of the audience filter when wrapped in web-mobile. Fix INBOX-4398
|
|
503
|
-
|
|
504
|
-
## Testing
|
|
505
|
-
- View the classification dashboard in the small browser window (mobile) in Lite. There should be a gap between the wrapped audience filter.
|
|
506
|
-
Also, the breadcrumb should only show one item, the parent level of the current topic.
|
|
507
|
-
|
|
508
|
-
"
|
|
509
|
-
clickview,Clean up - Fix new release banner bg image position,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2902,"## Lite
|
|
510
|
-
|
|
511
|
-
### Code changes
|
|
512
|
-
- Fix content updates banner bg image position Fix INBOX-4381
|
|
513
|
-
|
|
514
|
-
## Testing"
|
|
515
|
-
clickview,Lite: Search topics ordering,Joshua Taylor,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2899,"## Lite
|
|
516
|
-
|
|
517
|
-
### Customer changes
|
|
518
|
-
- As a user in a multi-presentation school, if you search for a topic and get a result with the same name from >1 presentation, they will be ordered by the learning level of the presentation.
|
|
519
|
-
|
|
520
|
-
## Testing
|
|
521
|
-
- Login to an account with both ""AU Primary"" and ""AU Secondary"" presentations. Search for ""English"". In the results, the english topic from the primary presentation should be shown before the english topic from the secondary presentation."
|
|
522
|
-
clickview,Remove duplicate id and sort them in Flight key,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2898,"## Lite
|
|
523
|
-
|
|
524
|
-
### Code changes
|
|
525
|
-
- For the `hasCurriculumLinks` request, make the array of objectId unique and sort them in key so it can be cached more accurately. Fixes INBOX-4316.
|
|
526
|
-
|
|
527
|
-
## Testing
|
|
528
|
-
- In NP, the curriculum links should show when a video have it and work as usual."
|
|
529
|
-
clickview,Search results interactives,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2896,"## Lite
|
|
530
|
-
|
|
531
|
-
### Customer changes
|
|
532
|
-
- In search results of small screen, the interactive will show 3 question. Implements INBOX-4344.
|
|
533
|
-
|
|
534
|
-
### Code changes
|
|
535
|
-
- Show 3 question of interactives in search result in mobile.
|
|
536
|
-
|
|
537
|
-
## Testing
|
|
538
|
-
- In NP, find an interactive by using search, in mobile, there should have 3 question showing."
|
|
539
|
-
clickview,Add alert callbacks after save video details,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2892,"## Lite
|
|
540
|
-
|
|
541
|
-
### Customer changes
|
|
542
|
-
- Customer will get an alert message after update video details. Implements INBOX-4116.
|
|
543
|
-
|
|
544
|
-
### Code changes
|
|
545
|
-
- Add alerts on callbacks after saving video details.
|
|
546
|
-
|
|
547
|
-
## Shared
|
|
548
|
-
|
|
549
|
-
### Code changes
|
|
550
|
-
- `updateVideo` request accepts `always` callback.
|
|
551
|
-
|
|
552
|
-
## Testing
|
|
553
|
-
- In NP, edit a video in 'My Videos', after save the changes, an alert should show and say whether it is successfully of failed to update video details."
|
|
554
|
-
clickview,Update copy of student empty state,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2891,"## Lite
|
|
555
|
-
|
|
556
|
-
### Customer changes
|
|
557
|
-
- When student has no contents, the empty state copy has been updated. Implements INBOX-4291.
|
|
558
|
-
|
|
559
|
-
### Code changes
|
|
560
|
-
- Update copy of empty state in student feed.
|
|
561
|
-
|
|
562
|
-
## Testing
|
|
563
|
-
- In NP, login as a student without content and go to student feed, the copy should be updated."
|
|
564
|
-
clickview,Reorder action button in interactive preview page,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2888,"## Lite
|
|
565
|
-
|
|
566
|
-
### Code changes
|
|
567
|
-
- Reorder the `Share` button after the `Add to playlist` button in the lite interactive page. Fix INBOX-4334
|
|
568
|
-
|
|
569
|
-
## Testing
|
|
570
|
-
- In lite, preview an interactive, the button `Share` should be after `Add to playlist` button."
|
|
571
|
-
clickview,Cross platform build,Cameron Hill,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2878,"## Monorepo
|
|
572
|
-
|
|
573
|
-
### Code changes
|
|
574
|
-
- Cleanup commands for cross platform builds. Closes INBOX-4286
|
|
575
|
-
- Remove all docker build scripts
|
|
576
|
-
- Replace `export` and `set` environment variable calls with `cross-env` package to enable cross platform building without needing separate linux only commands
|
|
577
|
-
|
|
578
|
-
## Testing
|
|
579
|
-
- All projects build on windows and linux"
|
|
580
|
-
Share.Api,Using parent video and clip virtual assoc,Prad Nepal,Kevin Wellalagodage,https://gitlab.cvinternal.net/clickview/back-end/share.api/-/merge_requests/73,"## Dependencies
|
|
581
|
-
* [ ] Needs to be pushed after Objects API Z1 release
|
|
582
|
-
|
|
583
|
-
|
|
584
|
-
## Code changes
|
|
585
|
-
* Added parent video virtual assoc to the interactive and clip
|
|
586
|
-
|
|
587
|
-
|
|
588
|
-
## Testing
|
|
589
|
-
`_parentVideo` and `_parentClip` properties must exist for shared by me and feed responses"
|
|
590
|
-
Search.Api,Parent video and clip,Prad Nepal,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/search.api/-/merge_requests/130,"## Dependencies
|
|
591
|
-
* [ ] Needs to be pushed after Objects API Z1 release has been pushed
|
|
592
|
-
|
|
593
|
-
|
|
594
|
-
## Code changes
|
|
595
|
-
* Added parent video virtual assoc to the interactive and clip
|
|
596
|
-
|
|
597
|
-
|
|
598
|
-
## Testing
|
|
599
|
-
* `_parentVideo` and `_parentClip` properties must exist in the search result for interactive and clip"
|
|
600
|
-
Search.Api,Playlist filter,Prad Nepal,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/search.api/-/merge_requests/129,"## Code changes
|
|
601
|
-
* Added playlist filter. Fixes INBOX-4085
|
|
602
|
-
|
|
603
|
-
|
|
604
|
-
## Testing
|
|
605
|
-
* Should be able to filter videos by playlist ids by passing `playlist` filter in the query parameter"
|
|
606
|
-
Search.Api,Clip int audiences,Choco Li,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/search.api/-/merge_requests/128,"## Code changes
|
|
607
|
-
- Add audience to video in clip and interactive
|
|
608
|
-
|
|
609
|
-
|
|
610
|
-
## Testing"
|
|
611
|
-
Ship It,Fix DefaultProjectConfig being altered by _.merge,Cameron Hill,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/ship-it/-/merge_requests/44,"## Code changes
|
|
612
|
-
- Fixed ship-it default project config being modified by `_.merge()`
|
|
613
|
-
|
|
614
|
-
|
|
615
|
-
## Testing"
|
|
616
|
-
Ship It,Project config + Other stuff,Cameron Hill,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/ship-it/-/merge_requests/42,"## Code changes
|
|
617
|
-
- Added support for `.ship-it.config` project config files in project directory. Fixes INBOX-4378
|
|
618
|
-
- Add a `.ship-it.config` file to the root of a project to configure ship it for that project. Current settings include:
|
|
619
|
-
- `monorepo.enabled` = Enables monorepo support
|
|
620
|
-
- `monorepo.validSubProjects` = An array of valid sub projects which is used for section validation in MR descriptions
|
|
621
|
-
- Changed description validation logic
|
|
622
|
-
- Description sections are now case sensitive
|
|
623
|
-
- The valid sub project sections are controlled by the project config
|
|
624
|
-
- Removed support for adding unknown sections in the MR description
|
|
625
|
-
- Added a heading transformer to fix casing issues of known headings
|
|
626
|
-
- Added error output to description validation
|
|
627
|
-
- Updated release note heading ordering logic
|
|
628
|
-
- monorepo projects are now ordered alphabetically
|
|
629
|
-
- Notes are now at the bottom
|
|
630
|
-
|
|
631
|
-
|
|
632
|
-
## Testing
|
|
633
|
-
- Run for Y2 release and compare the results with and without these changes. They should be mostly the same (except some MR validation errors because of heading case changes)"
|
|
634
|
-
Discover.Api,Using parent video and clip,Prad Nepal,Jorge Arai,https://gitlab.cvinternal.net/clickview/back-end/discover.api/-/merge_requests/103,"## Dependencies
|
|
635
|
-
* [ ] Should be pushed after Objects API Z1 release has been pushed
|
|
636
|
-
|
|
637
|
-
|
|
638
|
-
## Code changes
|
|
639
|
-
* Added `_parentVideo` and `_parentClip` virtual assoc for both clip and interactive
|
|
640
|
-
|
|
641
|
-
|
|
642
|
-
## Testing
|
|
643
|
-
* All clip and interactive should have `_parentVideo` and `_parentClip` virtual assocs"
|
|
644
|
-
Discover.Api,"Copy dashboard endpoint, implments CORE-1065",Wei Zong,Jorge Arai,https://gitlab.cvinternal.net/clickview/back-end/discover.api/-/merge_requests/102,"## Code changes
|
|
645
|
-
- Dashboard added new action method to copy dashboard and widgets
|
|
646
|
-
|
|
647
|
-
|
|
648
|
-
## Testing
|
|
649
|
-
- Find a dashboard id from `dashboards` tbl, POST to `v1/dashboards/{id}` endpoint, it should crate a new record in the `dashboards` tbl and a few records in the `dashboard_widgets`. Verify the new and old dashboards have the same dashboard widgets."
|
|
650
|
-
Discover.Api,Provide links to video responses,Jorge Arai,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/discover.api/-/merge_requests/99,"## Code changes
|
|
651
|
-
* Added `links` to video's ogql. Implements LTE-1754
|
|
652
|
-
|
|
653
|
-
|
|
654
|
-
## Testing
|
|
655
|
-
* In Discover Api, when fetching video content from widgets, property `links` should be available"
|
|
656
|
-
FileGenerator,Create TaskLogScope and use that to create a log scope,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/filegenerator/-/merge_requests/101,"## Code changes
|
|
657
|
-
- Added `TaskLogScope` to be used to create a log scope for each process task
|
|
658
|
-
- Add a `TaskId` for each task so we can group logs together to help diagnose issues like INBOX-4386
|
|
659
|
-
|
|
660
|
-
|
|
661
|
-
## Testing
|
|
662
|
-
- `TaskKey` and `TaskId` are populated for logs in Seq"
|
|
663
|
-
FileGenerator,Remove ToList() for email attachments,Cameron Hill,Kevin Wellalagodage,https://gitlab.cvinternal.net/clickview/back-end/filegenerator/-/merge_requests/100,"## Code changes
|
|
664
|
-
- Remove extra `ToList()` from all `ToArray()` calls when adding email attachments to reduce memory usage
|
|
665
|
-
|
|
666
|
-
|
|
667
|
-
## Testing
|
|
668
|
-
- Emails generated from FileGenerator (ie Content Reports) should have attachments that work"
|
|
669
|
-
Clients,Add ship-it config,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/67,"## monorepo
|
|
670
|
-
### Code changes
|
|
671
|
-
- Add `ship-it.config` file for latest changes in ship it https://gitlab.cvinternal.net/clickview/front-end/ship-it/-/merge_requests/42
|
|
672
|
-
|
|
673
|
-
|
|
674
|
-
## Testing
|
|
675
|
-
- Ship it runs"
|
|
676
|
-
Clients,MasterApi test cleanup,Cameron Hill,Jorge Arai,https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/66,"## ClickView.Clients.Master
|
|
677
|
-
### Code changes
|
|
678
|
-
- Remove `morelinq` and use .NET `Chunk()` extension method
|
|
679
|
-
- Cleanup tests and re-order them based on the order they exist in the interface
|
|
680
|
-
- Added tests around checking methods return null/empty when objects cannot be found
|
|
681
|
-
- Move client creation to a `CreateClient()` method
|
|
682
|
-
|
|
683
|
-
|
|
684
|
-
## Testing"
|
|
685
|
-
Clients,Added UpdateObject without a return type,Cameron Hill,Jorge Arai,https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/65,"## ClickView.Clients.Master
|
|
686
|
-
### Code changes
|
|
687
|
-
- Added `UpdateObjectAsync` without a type parameter that returns nothing
|
|
688
|
-
|
|
689
|
-
|
|
690
|
-
## Testing"
|
|
691
|
-
Clients,Add `GetPublicPermissionsAsync` to PermissionApiClient,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/64,"## ClickView.Clients.Permission
|
|
692
|
-
### Code changes
|
|
693
|
-
- Added `GetPublicPermissionsAsync()`
|
|
694
|
-
- Added missing `[Obsolete]` attribute for `GetUserPermissionsAsync(ulong userId)` on `IPermissionApiClient` (was on the base class but not the interface)
|
|
695
|
-
|
|
696
|
-
|
|
697
|
-
## Testing"
|
|
698
|
-
Clients,Add Master Api endpoints,Jorge Arai,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/62,"## ClickView.Clients.Master
|
|
699
|
-
### Code changes
|
|
700
|
-
* Added new methods to Clients.Master
|
|
701
|
-
* GetUsersByUsernameAsync
|
|
702
|
-
* GetUsersByEmailAsync
|
|
703
|
-
* GetRootMemberObjectsAsync
|
|
704
|
-
Implements INBOX-4368
|
|
705
|
-
|
|
706
|
-
|
|
707
|
-
## Testing
|
|
708
|
-
* N/A"
|
|
709
|
-
Lite,Update subject api endpoints used for curriculum links,Michael McDonnell,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/348,"## Dependencies
|
|
710
|
-
https://gitlab.cvinternal.net/clickview/front-end/clickview/-/merge_requests/2968
|
|
711
|
-
|
|
712
|
-
|
|
713
|
-
## Code changes
|
|
714
|
-
- Add `GetObjectClassificationPaths` endpoint to get an object's curriculum links
|
|
715
|
-
- Add `CheckObjectExistsInPresentations` endpoint to check if an object has curriculum links
|
|
716
|
-
|
|
717
|
-
|
|
718
|
-
## Testing
|
|
719
|
-
- N/A"
|
|
720
|
-
Lite,Run all boostrap handlers in parallel,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/347,"## Code changes
|
|
721
|
-
- Run all `IBootstrapHandler` in parallel to improve the initial page load times
|
|
722
|
-
|
|
723
|
-
|
|
724
|
-
## Testing
|
|
725
|
-
- Lite loads with bootstrap data in `window.bootstrapData`"
|
|
726
|
-
Lite,Bootstrap permissions and products,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/346,"## Before release
|
|
727
|
-
- [ ] Update `ClickView.Clients.Permission` to 2.10.0
|
|
728
|
-
|
|
729
|
-
|
|
730
|
-
## Dependencies
|
|
731
|
-
- https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/64
|
|
732
|
-
|
|
733
|
-
|
|
734
|
-
## Customer changes
|
|
735
|
-
- Improved performance of the initial page load for all regions
|
|
736
|
-
|
|
737
|
-
|
|
738
|
-
## Code changes
|
|
739
|
-
- Added bootstrap handlers for products and permissions. Implements LTE-1783
|
|
740
|
-
- Replace existing permission api passthrough with the bootstrap permission handler
|
|
741
|
-
- Moved `GetRoutePrefix` to be an extension method for `HttpRequest`
|
|
742
|
-
|
|
743
|
-
|
|
744
|
-
## Testing
|
|
745
|
-
- Permissions and products should exist in `window.bootstrapData` with the correct data types:
|
|
746
|
-
- Permissions should be an array of strings
|
|
747
|
-
- Products should be an array of product objects"
|
|
748
|
-
Lite,Always reading language file from disk in development,Shale Kuzmanovski,Cameron Hill,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/345,"## Code changes
|
|
749
|
-
- The `PhraseHandler` bootstrap handler will now always read from disk in development, allowing the team to make language changes without having to restart the dotnet project. Fixes INBOX-4460.
|
|
750
|
-
|
|
751
|
-
|
|
752
|
-
## Testing"
|
|
753
|
-
Lite,"Add new templates for ""select with spreadsheet""",Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/343,"## Code changes
|
|
754
|
-
- Add a new template for ""Select with spreadsheet"" in the user manager.
|
|
755
|
-
|
|
756
|
-
|
|
757
|
-
## Testing"
|
|
758
|
-
Lite,Update to proxied staging braze sdk urls,Nischal Shrestha,Shale Kuzmanovski,https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/342,"## Before release
|
|
759
|
-
- Update Braze sdk urls in octo to the proxied prod urls
|
|
760
|
-
|
|
761
|
-
|
|
762
|
-
## Code changes
|
|
763
|
-
- Update braze sdk urls in `appsettings.json`
|
|
764
|
-
|
|
765
|
-
|
|
766
|
-
## Testing
|
|
767
|
-
- N/A"
|
|
768
|
-
Master.Auth,Nullable fixes,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/356,"## Code changes
|
|
769
|
-
- Nullable warning fixes
|
|
770
|
-
- Clever `DistrictAdmin.Title` is now nullable
|
|
771
|
-
- Added `IUrlHelper.RouteUri` extension method which returns a non nullable Uri
|
|
772
|
-
- Throw an exception in `CertificateLoader.ReadFromThumbprint` if the certification cannot be found (matches `LoadFromFile`)
|
|
773
|
-
- Fix string check in `LockoutRepository.GetLockoutEndAsync`
|
|
774
|
-
- Add `MaybeNullWhen` to `ActivateAjaxController.TryValidateRequest`
|
|
775
|
-
- Add `MemberNotNullWhen` to `LoggedOutViewModel.TriggerExternalSignOut`
|
|
776
|
-
- Use `CreatedAtRoute` instead of `Url.Link` in `POST /v1/clients`
|
|
777
|
-
- MasterApiClient returns nullable objects on 404
|
|
778
|
-
- Apparently `IMemoryCache.GetOrCreateAsync()` can return null even when not marked as null
|
|
779
|
-
|
|
780
|
-
|
|
781
|
-
## Testing
|
|
782
|
-
- When creating a new auth client, the 201 contains the correct location in the header and the body is the client id
|
|
783
|
-
- The link in the activation email should be `https` and the correct URL"
|
|
784
|
-
Master.Auth,Replace masterclient,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/355,"## Before release
|
|
785
|
-
- Update `ClickView.Clients.Master` to `2.10.0`
|
|
786
|
-
|
|
787
|
-
|
|
788
|
-
## Dependencies
|
|
789
|
-
- https://gitlab.cvinternal.net/clickview/back-end/clients/-/merge_requests/65
|
|
790
|
-
|
|
791
|
-
|
|
792
|
-
## Code changes
|
|
793
|
-
- Replace local `MasterApiClient` with `ClickView.Clients.Master`
|
|
794
|
-
|
|
795
|
-
|
|
796
|
-
## Testing
|
|
797
|
-
- Can login"
|
|
798
|
-
Master.Auth,Update mediatr,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/354,"## Code changes
|
|
799
|
-
- Update MediatR to the latest version
|
|
800
|
-
- Remove DI package (no longer needed)
|
|
801
|
-
- Implement `INotificationPublisher` instead of a custom `MediatR` class
|
|
802
|
-
|
|
803
|
-
|
|
804
|
-
## Testing
|
|
805
|
-
- When logging in, the users last sign in time should be updated"
|
|
806
|
-
Master.Auth,Authentication restructure,Cameron Hill,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/353,"## Code changes
|
|
807
|
-
- Restructure the authenticator folders
|
|
808
|
-
|
|
809
|
-
|
|
810
|
-
## Testing"
|
|
811
|
-
Master.Auth,User analytics,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/351,"## Code changes
|
|
812
|
-
* Populating year group and user group properties to user analytics object. Fixes INBOX-4342
|
|
813
|
-
|
|
814
|
-
|
|
815
|
-
## Testing
|
|
816
|
-
* `cv_product_auth_login` events should have `userGroup` and `yearGroup` properties populated"
|
|
817
|
-
ClickView.EmailService,MailerService updates,Cameron Hill,Kevin Wellalagodage,https://gitlab.cvinternal.net/clickview/back-end/ClickView.EmailService/-/merge_requests/222,"## Code changes
|
|
818
|
-
- Fix dependencies having direct object references instead of the method creation func
|
|
819
|
-
- Dependencies should be disposed on shutdown now
|
|
820
|
-
- Small code cleanups
|
|
821
|
-
- Remove #region
|
|
822
|
-
- Move private methods under public methods
|
|
823
|
-
- Change `Attachment.Bytes` to be a byte array instead of a list of bytes
|
|
824
|
-
- Avoids the extra memory allocations converting Lists to Arrays etc
|
|
825
|
-
- Remove `ForEach(async)` in `MailerService` and replace with a normal `foreach`
|
|
826
|
-
- Update `MailerService.AddAttachment` to copy to a local memory stream so the HttpClient's response can be disposed correctly
|
|
827
|
-
|
|
828
|
-
|
|
829
|
-
## Testing
|
|
830
|
-
- Sending emails with attachments should work (make sure they are attachments and not links to the files)
|
|
831
|
-
- Sending emails with the `UrlAttachments` property populated should have those files added to the email as attachments"
|
|
832
|
-
ClickView Online,Add QueryUsers endpoint,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Online/-/merge_requests/3194,"## Code changes
|
|
833
|
-
- Add QueryUsers endpoint
|
|
834
|
-
|
|
835
|
-
|
|
836
|
-
## Testing"
|
|
837
|
-
ClickView Online,Add new template for select with spreadsheet,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Online/-/merge_requests/3193,"## Code changes
|
|
838
|
-
- Add new SelectWithSpreadsheet template
|
|
839
|
-
|
|
840
|
-
|
|
841
|
-
## Testing"
|
|
842
|
-
ClickView Online,Add student to download allowed roles,Jorge Arai,Cameron Hill,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Online/-/merge_requests/3190,"## Customer changes
|
|
843
|
-
* Students should be able to download the videos from their workspace
|
|
844
|
-
|
|
845
|
-
|
|
846
|
-
## Code changes
|
|
847
|
-
* Added `Student` to `OnlineConfig.config > Download > roles`. Fixes INBOX-4395
|
|
848
|
-
|
|
849
|
-
|
|
850
|
-
## Testing
|
|
851
|
-
* In Online, students should be able to download videos from their own workspace"
|
|
852
|
-
ClickView Online,Fix the logic to identify if a video is from exchange,Choco Li,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Online/-/merge_requests/3188,"## Customer changes
|
|
853
|
-
- The copy of confirm popup when creating interactive from a video from exchange is updated to say 'Would you like other teachers on the Exchange to be able to see your interactive for {videoName}? Fixes INBOX-4251.
|
|
854
|
-
|
|
855
|
-
|
|
856
|
-
## Code changes
|
|
857
|
-
- Fix the logic to identify if a video is from exchange.
|
|
858
|
-
|
|
859
|
-
|
|
860
|
-
## Testing
|
|
861
|
-
- Find a video in exchange and create an interactive from it, add any question(s) then keep clicking 'Save and continue' until `Publish your interactive` popup shows. The popup should say 'Would you like other teachers on the Exchange to be able to see your interactive for {videoName}?"
|
|
862
|
-
ClickView.Curator,Add QueryUsers to user api,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1056,"## Code changes
|
|
863
|
-
- Add QueryUser to user api
|
|
864
|
-
|
|
865
|
-
|
|
866
|
-
## Testing"
|
|
867
|
-
ClickView.Curator,Add template for user-man select with spreadsheet,Nan Chen,Joshua Taylor,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1054,"## Code changes
|
|
868
|
-
- Add new SelectWithSpreadsheet template
|
|
869
|
-
|
|
870
|
-
|
|
871
|
-
## Testing"
|
|
872
|
-
ClickView.Curator,Split ajax controller,Cameron Hill,Wei Zong,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1051,"## Customer changes
|
|
873
|
-
- Improve curator performance by ~100% to ~300%
|
|
874
|
-
|
|
875
|
-
|
|
876
|
-
## Code changes
|
|
877
|
-
- Split the `AjaxController` into individual controllers instead of one big partial controller. Closes INBOX-4471
|
|
878
|
-
- Reduces the number of services needed to be created in the DI for every ajax request
|
|
879
|
-
|
|
880
|
-
|
|
881
|
-
## Testing
|
|
882
|
-
- General smoke test of curator (every ajax endpoint was updated)"
|
|
883
|
-
ClickView.Curator,Improve /ajax/playlists speed,Cameron Hill,Wei Zong,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1049,"## Code changes
|
|
884
|
-
- Improve performance of mongodb connections
|
|
885
|
-
- Only use a single instance of the `MongoClient` instead of creating one for every request.
|
|
886
|
-
- Added `MongoConnectionFactory` to handle MongoDb connections
|
|
887
|
-
- Mark all mongodb repositories as singleton
|
|
888
|
-
- Mark MasterApi clients as Singleton
|
|
889
|
-
- Improve loading speed of `/ajax/playlists/{id}` by fetching the playlist and the mongodb data in parallel. Fixes INBOX-3708
|
|
890
|
-
|
|
891
|
-
|
|
892
|
-
## Testing
|
|
893
|
-
- Loading playlists on the `/dashboards/{id}` page should be a little faster than 13s"
|
|
894
|
-
ClickView.Curator,Show subtitle section regardless chapter data,Choco Li,Michael McDonnell,https://gitlab.cvinternal.net/clickview/front-end/ClickView.Curator/-/merge_requests/1044,
|
|
895
|
-
Subjects.API,Add v2/objects/exist,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/clickview.subjects.api/-/merge_requests/189,"## Code changes
|
|
896
|
-
- Added `/v2/objects/exists` as a v2 alternative to `/v1/customers/{customerid}/users/{userid}/presentations/objects`
|
|
897
|
-
- Update `ExistsInPresentationsAsync` to check if the first object exists instead of using count to hopefully reduce load on master api
|
|
898
|
-
|
|
899
|
-
|
|
900
|
-
## Testing"
|
|
901
|
-
Subjects.API,Fix HashIdExtensions.TryDecodeUlong throwing exceptions,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/clickview.subjects.api/-/merge_requests/188,"## Customer changes
|
|
902
|
-
- Fixes an issue where fetching a classification by its id would sometimes throw a 500 if the id was unknown
|
|
903
|
-
|
|
904
|
-
|
|
905
|
-
## Code changes
|
|
906
|
-
- Don't throw exceptions in `HashIdExtensions.TryDecodeUlong`
|
|
907
|
-
|
|
908
|
-
|
|
909
|
-
## Testing
|
|
910
|
-
- `GET /v2/classifications/aaaa` should return 404 not 500"
|
|
911
|
-
ClickView.Cron,Lite upgrade task fix,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/ClickView.Cron/-/merge_requests/27,"## Code changes
|
|
912
|
-
* Was missing lite upgrade task registration. Fixes INBOX-4317
|
|
913
|
-
|
|
914
|
-
|
|
915
|
-
## Testing
|
|
916
|
-
* Make sure `Lite upgrade task` runs on schedule"
|
|
917
|
-
Domain.Api,Send playlist id to search api,Prad Nepal,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/Domain.Api/-/merge_requests/1075,"## Dependencies
|
|
918
|
-
* Merge after https://gitlab.cvinternal.net/clickview/back-end/search.api/-/merge_requests/129
|
|
919
|
-
|
|
920
|
-
|
|
921
|
-
## Code changes
|
|
922
|
-
* Sending playlist ids as to search API as filter. Fixes INBOX-4085
|
|
923
|
-
|
|
924
|
-
|
|
925
|
-
## Testing
|
|
926
|
-
* `Search in this playlist` feature in online should work"
|
|
927
|
-
Domain.Api,Remove log4net,Ryan Wang,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/Domain.Api/-/merge_requests/1074,"## Code changes
|
|
928
|
-
* Get rid of `Log4Net` logging as it is no longer in use for production
|
|
929
|
-
* Update non-production configs to use staging seq for logging
|
|
930
|
-
|
|
931
|
-
|
|
932
|
-
## Testing
|
|
933
|
-
* Everything should be still in one piece"
|
|
934
|
-
Domain.Api,Cleanups,Ryan Wang,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/Domain.Api/-/merge_requests/1073,"## Code changes
|
|
935
|
-
* Upgrade the nuget packages to their latest if they can be upgraded
|
|
936
|
-
* Minor code clean up to fix up some warnings
|
|
937
|
-
|
|
938
|
-
|
|
939
|
-
## Testing
|
|
940
|
-
* Regression test to ensure that everything should be still working"
|
|
941
|
-
Domain.Api,Share bug,Ryan Wang,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/Domain.Api/-/merge_requests/1071,"## Customer changes
|
|
942
|
-
* Fixed an issue where playlist share links do not work after domain migration. Close INBOX-4320
|
|
943
|
-
|
|
944
|
-
|
|
945
|
-
## Code changes
|
|
946
|
-
* Replace `CustomerId` and `EmailAddress` with `MasterCustomerId` and `UserId` in `SharedResource`
|
|
947
|
-
* Look up sharer's user context by using `MasterCustomerId` and `UserId` instead of customer guid and username
|
|
948
|
-
* Clean up code
|
|
949
|
-
|
|
950
|
-
|
|
951
|
-
## Testing
|
|
952
|
-
* In Online, create a school-only share link of a playlist or use the existing playlist share link
|
|
953
|
-
* Go to shortcode api db to change the `EmailAddress` to something else of that playlist sharecode to mimic the domain migration
|
|
954
|
-
* Use another account from the same customer to open the share link, the shared playlist should be loaded"
|
|
955
|
-
Objects.Api,Archived video interactives fix,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/objects.api/-/merge_requests/232,"## Code changes
|
|
956
|
-
* Updated the code to allow users to access interactives in archived videos. Fixes INBOX-4425
|
|
957
|
-
|
|
958
|
-
|
|
959
|
-
## Testing
|
|
960
|
-
* On online, create an interactive on an archived video and go to the video play page. In the interactive tab, the interactive should show up"
|
|
961
|
-
Objects.Api,Playlist reorder fix,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/objects.api/-/merge_requests/231,"## Code changes
|
|
962
|
-
* Currently users can see both archived and published videos in playlists but can't reorder archived videos. This change fixes it. Fixes INBOX-4404
|
|
963
|
-
|
|
964
|
-
|
|
965
|
-
## Testing
|
|
966
|
-
* Add a video to a playlist and then archive it. You should still be able to see the video in the playlist and reorder the video."
|
|
967
|
-
Objects.Api,Fixed ogql optimiser,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/objects.api/-/merge_requests/230,"## Code changes
|
|
968
|
-
* If the resources, links, etc was shared then use sharer's permision values to perform the read checks. Fixes INBOX-4356
|
|
969
|
-
|
|
970
|
-
|
|
971
|
-
## Testing
|
|
972
|
-
* When content is shared, links and resources should be available to the shared user"
|
|
973
|
-
Objects.Api,Virtual Assoc Timeout,Prad Nepal,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/objects.api/-/merge_requests/228,"## Code changes
|
|
974
|
-
* Added `BaseTimeLimitVirtualAssocHandler` which can be inherited by any virtual assoc that needs to timeout after a certain period. Implements INBOX-4187
|
|
975
|
-
* Default timeout is set to 1.5 seconds
|
|
976
|
-
* Any virtual assoc that takes more than the timeout period will log to seq and return null
|
|
977
|
-
|
|
978
|
-
|
|
979
|
-
## Testing
|
|
980
|
-
* Virtual assoc that takes more than 1.5 seconds should timeout and return null"
|
|
981
|
-
Objects.Api,Parent video and clip virtual assoc,Prad Nepal,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/objects.api/-/merge_requests/227,"## Code changes
|
|
982
|
-
* Added virtual assoc hander to get parent video and clip of an interactive
|
|
983
|
-
* Added virtual assoc handler to get parent video of a clip
|
|
984
|
-
* Added mapping for the new virtual assoc handlers
|
|
985
|
-
* Returning virtual assoc as null if the data is null
|
|
986
|
-
* Made result consistent between single and array of virtual assocs
|
|
987
|
-
|
|
988
|
-
|
|
989
|
-
## Testing
|
|
990
|
-
* Sending `_parentVideo` or `_parentClip` should return video and clip that the interactive or clip was created on"
|
|
991
|
-
Krakend,"Added user api endpoint to retrieve user ids by list of username and emails, implments CORE-1073",Wei Zong,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/krakend/-/merge_requests/97,"## Code changes
|
|
992
|
-
- Added user api endpoint to retrieve user ids by list of username and emails
|
|
993
|
-
|
|
994
|
-
|
|
995
|
-
## Testing
|
|
996
|
-
- Pass on list of emails and list of usernames to `GET v1/users/ids` endpoint, it should return list of unique user ids"
|
|
997
|
-
Users.Api,"Add BulkGetUserIds action method, partially implments CORE-1064",Wei Zong,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/users.api/-/merge_requests/157,"## Before release
|
|
998
|
-
- Add new route mapping to the Krakend and release Kranend in the same time
|
|
999
|
-
|
|
1000
|
-
|
|
1001
|
-
## Code changes
|
|
1002
|
-
- UserController adds `GET v1/users/ids` endpoint with optional `usertype` and `customerIdOverride` arguments, which reads list of emails and list of usernames and returns list of unique user ids belonging to the school
|
|
1003
|
-
|
|
1004
|
-
|
|
1005
|
-
## Testing
|
|
1006
|
-
- Log in using an account `with curator access`. Pass on list of emails and list of usernames to `GET v1/users/ids` endpoint, it should return list of unique user ids.
|
|
1007
|
-
- Log in using an account `without curator access`.Pass on list of emails and list of usernames to `GET v1/users/ids` endpoint, it should return list of unique user ids.
|
|
1008
|
-
- Try to get a big list of of email and username and ensure all the user ids are returned."
|
|
1009
|
-
Users.Api,Always send activation if email provided,Ryan Wang,Cameron Hill,https://gitlab.cvinternal.net/clickview/back-end/users.api/-/merge_requests/155,"## Customer changes
|
|
1010
|
-
* Address an issue where students are unable to sign in when they are created with provided email but not provided with credentials by the admin. Close CORE-1060
|
|
1011
|
-
|
|
1012
|
-
|
|
1013
|
-
## Code changes
|
|
1014
|
-
* Reset the password to `null` if an email is provided when creating a user regardless of the user type
|
|
1015
|
-
* Do not need to auto generate password if email is provided even though `GeneratePassword` is set to `true` in the `CreateUserRequest`
|
|
1016
|
-
|
|
1017
|
-
|
|
1018
|
-
## Testing
|
|
1019
|
-
* In user manager, create a student account with an email, an activation email should be sent to the provided email
|
|
1020
|
-
* In user manager, create a student account without an email, the credentials should appear on the UI for shareing with student
|
|
1021
|
-
* In user manager, upload a spreadsheet to create students in bulk with emails set for each student, each student should receive an activation email to activate their account by themselves
|
|
1022
|
-
* In user manager, upload a spreadsheet to create student in bulk with emails set for some of students and without setting password for any student, activation email should be sent to those students that have emails set. For those that have no email, the passwords are automatically created for them and the credentials are included in the `Create users bulk action report` csv file."
|
|
1023
|
-
Permissions.Api,Replace local master api client with Clients.Master,Cameron Hill,Jorge Arai,https://gitlab.cvinternal.net/clickview/back-end/permissions.api/-/merge_requests/32,"## Code changes
|
|
1024
|
-
- Replace local master api client with `ClickView.Clients.Master`
|
|
1025
|
-
|
|
1026
|
-
|
|
1027
|
-
## Testing
|
|
1028
|
-
- When a customer is moved to a different access policy that disables the customer, the customer is disabled in master api
|
|
1029
|
-
- Api returns correct products for a customer (online shou;d have the correct url region for the instance region of the customer)
|
|
1030
|
-
- Api returns correct permissions for a user"
|
|
1031
|
-
VideoProcessorService,Update all packages,Cameron Hill,Ryan Wang,https://gitlab.cvinternal.net/clickview/back-end/VideoProcessorService/-/merge_requests/109,"## Code changes
|
|
1032
|
-
- Update all packages
|
|
1033
|
-
|
|
1034
|
-
|
|
1035
|
-
## Testing
|
|
1036
|
-
- General smoke test of processing videos
|
|
1037
|
-
- Processes the video
|
|
1038
|
-
- Uploads it to aws
|
|
1039
|
-
- generates a thumbnail"
|
|
1040
|
-
Image.Api,Remove puppeteer from image api,Cameron Hill,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/Image.Api/-/merge_requests/103,"## Before release
|
|
1041
|
-
- Remove `ChromePath` and `BrowserLifetime` from octo
|
|
1042
|
-
|
|
1043
|
-
|
|
1044
|
-
## Code changes
|
|
1045
|
-
- `/v1/previews` will no longer return previews for pdfs
|
|
1046
|
-
- Remove Puppeteer this is no longer needed in image api
|
|
1047
|
-
- Remove template service which was used in the screenshot services (remove)
|
|
1048
|
-
|
|
1049
|
-
|
|
1050
|
-
## Testing
|
|
1051
|
-
- The resources in NP have thumbnails/preview images. In staging this will only happen for new resources since we didnt generate them for the old ones"
|
|
1052
|
-
RealtimeSalesforceSync,Replace code with GoodStuff.EventHandling.Master,Jorge Arai,Kevin Wellalagodage,https://gitlab.cvinternal.net/clickview/back-end/RealtimeSalesforceSync/-/merge_requests/81,"## Before release
|
|
1053
|
-
- Update `GoodStuff.EventHandling.Master` with the latest released version
|
|
1054
|
-
|
|
1055
|
-
|
|
1056
|
-
## Dependencies
|
|
1057
|
-
- Merge after https://gitlab.cvinternal.net/back-end/goodstuff-internal/-/merge_requests/14
|
|
1058
|
-
|
|
1059
|
-
|
|
1060
|
-
## Code changes
|
|
1061
|
-
- Master event codes replaced with `GoodStuff.EventHandling.Master`. Implements INBOX-3289
|
|
1062
|
-
|
|
1063
|
-
|
|
1064
|
-
## Testing
|
|
1065
|
-
- RealtimeSalesforceSync should work as expected"
|
|
1066
|
-
Prospect.Api,Add auth,Kevin Wellalagodage,Prad Nepal,https://gitlab.cvinternal.net/clickview/back-end/Prospect.Api/-/merge_requests/4,"## Dependencies
|
|
1067
|
-
- https://gitlab.cvinternal.net/clickview/back-end/Master.Auth/-/merge_requests/343
|
|
1068
|
-
- Clients
|
|
1069
|
-
- https://gitlab.cvinternal.net/clickview/front-end/lite/-/merge_requests/333
|
|
1070
|
-
- https://gitlab.cvinternal.net/clickview/back-end/users.api/-/merge_requests/150
|
|
1071
|
-
|
|
1072
|
-
|
|
1073
|
-
## Code changes
|
|
1074
|
-
- Require `internal.prospect.read` for read endpoints and `internal.prospect.write` for write endpoints
|
|
1075
|
-
|
|
1076
|
-
|
|
1077
|
-
## Testing
|
|
1078
|
-
- Ensure a 403 is returned when calling any Prospect API endpoints without the `internal.prospect.read` or `internal.prospect.write` scopes
|
|
1079
|
-
- Ensure both write and read Prospect API endpoints work when using the `internal.prospect.read` or `internal.prospect.write` scopes"
|