Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate transactions on some views #641

Open
markbrough opened this issue Mar 7, 2022 · 5 comments
Open

Duplicate transactions on some views #641

markbrough opened this issue Mar 7, 2022 · 5 comments
Labels

Comments

@markbrough
Copy link

I'm not sure if this is generally used, but I normally found it a nice and easy way to quickly preview activities.

However, I just noticed that each transaction is displayed twice:
https://d-portal.org/q.html?aid=US-GOV-1-72066921LA00001

The source XML shows transactions only once:
https://d-portal.org/q.xml?aid=US-GOV-1-72066921LA00001

As does Datastore Classic:
https://datastore.codeforiati.org/api/1/access/activity.xml?iati-identifier=US-GOV-1-72066921LA00001

In this view, transactions are displayed (correctly) only once each:
https://d-portal.org/ctrack.html#view=act&aid=US-GOV-1-72066921LA00001

@xriss
Copy link
Collaborator

xriss commented Mar 8, 2022

That is strange, will take a look.

BTW the difference is one view is built on the server and the other is built in the browser and it probably got broken somehow when we switched to streaming which was a big rewrite of how results like this are generated.

@notshi
Copy link
Collaborator

notshi commented Mar 19, 2022

Thanks for raising this, Mark.

Should probably mention that we've made some changes to the SAVi view and it now has it's own url.

So for the example you mentioned, the updated url is
https://d-portal.org/savi/?aid=US-GOV-1-72066921LA00001

where q.html is replaced with savi/. You can still switch to the print-friendly theme if you prefer.

You can also view org files this way (loading times might differ depending on how large these can get).

Here is the discussion about it (link).

@markbrough
Copy link
Author

Oh cool, thanks a lot for the reply and clarification, guys! Do you think it might sense to remove / redirect q.html to avoid confusion?

@notshi
Copy link
Collaborator

notshi commented Mar 20, 2022

q is our (raw data) API location and returns a pure html page, where it was previously loading javascript to display the theme; ie. you can 'View source' here and not a SAVi page.

In essence, to remove/redirect q.extension would be to change the d-portal API location so not ideal :)

We've decided to display the print-friendly theme here to differentiate the raw data from the themed SAVi. Hope that makes sense.

Btw, you can view the SAVi version from q.html via the iati-identifier link.

@notshi notshi added the bug label Mar 21, 2022
@notshi
Copy link
Collaborator

notshi commented Mar 21, 2022

Hi @markbrough this fix is now live. Could you check and let me know otherwise?

It was indeed due to the streaming update that this triggered a double passing of the first activity. Graphs should also now work where it wasn't previously due to mixed content https issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants