Skip to content

Conversation

@ahtesham-quraish
Copy link
Contributor

@ahtesham-quraish ahtesham-quraish commented Dec 23, 2025

What are the relevant tickets?

After discussion with Chris we finalized the following thing https://github.com/mitodl/hq/issues/9684#issuecomment-3682869381

Description (What does it do?)

There are improvements we need to do like given below

  • Consolidate multiple calls into single one on reader side
  • Remove the dialog that is used for taking the input from user instead we should copy paste the resource url directly inside the editor and hit enter it will add the resource card node.

Screenshots (if appropriate):

Screen.Recording.2025-12-23.at.7.33.12.PM.mov

How can this be tested?

  • Need to insert the url of resource for local you need to insert something like this http://open.odl.local:8062/search?resource=297 and click enter it will add the resource card node in editor. On reader side you can refresh the page and see the network bulk api call which will have all the resource id.
  • On hover the resource card you would see the cross button at the right top corner of the resource card which will be used for delete.

Additional Context

@github-actions
Copy link

OpenAPI Changes

Show/hide 1 changes: 0 error, 0 warning, 1 info
1 changes: 0 error, 0 warning, 1 info
info	[endpoint-added] at head/openapi/specs/v1.yaml	
	in API GET /api/v1/learning_resources/bulk/
		endpoint added


Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@ahtesham-quraish ahtesham-quraish force-pushed the ahtesham/drawer-branch branch 2 times, most recently from 1021ede to ae34417 Compare December 23, 2025 11:59
@ahtesham-quraish ahtesham-quraish marked this pull request as ready for review December 24, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants