Skip to content

Conversation

@EnzoVezzaro
Copy link
Contributor

Changes proposed in this PR:

  • Reduce records from 25 to 8 per page
  • Add refresh button in pagination
  • handle last file upload (on the history list)
  • change upload logic: after upload, now users get the ddo link immediately (they don't have to ask for it)

@EnzoVezzaro EnzoVezzaro self-assigned this Nov 2, 2023
@vercel
Copy link

vercel bot commented Nov 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dbs-ui-lib ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 9, 2023 2:16pm

@EnzoVezzaro
Copy link
Contributor Author

EnzoVezzaro commented Nov 6, 2023

Hi @jamiehewitt15, can you review this please.

NOTE: Some things have changed on the upload. Pay attention to the ddl link that is being shown to the user right after the uplodad has finished.

@jamiehewitt15
Copy link
Contributor

Not sure exactly what's wrong but it didn't work
image

@EnzoVezzaro
Copy link
Contributor Author

EnzoVezzaro commented Nov 8, 2023

Mmmm.... what happened? 😞 I'm sure I tested this, but now I'm getting this error:

Error uploading to micro-service: Error occurred while making the request: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response')). Detailed message: No detailed message provided.

Did we changed anything on the arweave microservice that could break the upload?

UPDATE: also because I didn't changed anything regarding upload, this PR is just for the pagination and history list. I don't see anything in the code that could've broken the upload.

@jamiehewitt15
Copy link
Contributor

I'll have a look in the logs on the backend to see if I can see what's happening

@jamiehewitt15
Copy link
Contributor

Looks like the arweave microservice still isn't working in the test environment. I don't think it's anything to do with this PR though, I think it's related to a change on the backend

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.

History list improvements

3 participants