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

catalog as string becomes long unbreakable string #75

Open
volodymyrss opened this issue Oct 28, 2021 · 10 comments
Open

catalog as string becomes long unbreakable string #75

volodymyrss opened this issue Oct 28, 2021 · 10 comments
Assignees

Comments

@volodymyrss
Copy link
Member

curl "https://www.astro.unige.ch/mmoda/dispatch-data/inspect-state?token=$(cat ~/.oda-token | xargs)&job_id=62840af4b40af6c8" | jq

@volodymyrss volodymyrss self-assigned this Oct 28, 2021
@volodymyrss
Copy link
Member Author

unclear why catalog is passed as a string. if it was json would be ok. also could use this oda-hub/dispatcher-app#267

also, need to verify why exactly this was not breakable, it's not too bad

@burnout87
Copy link
Collaborator

Can you please grant me the role to run this query? I cannot see anything otherwise,

@volodymyrss
Copy link
Member Author

done. We should add a separate role though, in oda-hub/dispatcher-app#263

@burnout87
Copy link
Collaborator

unclear why catalog is passed as a string.

When used the functionality Use catalog, the catalog is string-fied by the fontend and passed as string as a whole as argument in the request.

@volodymyrss
Copy link
Member Author

unclear why catalog is passed as a string.

When used the functionality Use catalog, the catalog is string-fied by the fontend and passed as string as a whole as argument in the request.

and in the case of API?

@burnout87
Copy link
Collaborator

and in the case of API?

We once wrote a test for indeed testing the catalog sent via the API, and also in that case we passed the catalog as a string.

https://github.com/oda-hub/dispatcher-app/blob/c72ba4763afb9821a9f877d3d9fe06dba5a2b599/tests/test_server_basic.py#L982-L1042

@volodymyrss
Copy link
Member Author

ok then, the test should be expanded to inspect the email.

@burnout87
Copy link
Collaborator

Which is exactly what I am working on.

@volodymyrss volodymyrss added this to the v21.10.0002 milestone Nov 2, 2021
@volodymyrss
Copy link
Member Author

done. We should add a separate role though, in oda-hub/dispatcher-app#263

Note that since I added this role to you, you started to recieved user approval emails. Its circumstantial, it's really Carlo who knows most people who are likely to join, and sometimes you. We should really add a new role.

@burnout87
Copy link
Collaborator

done. We should add a separate role though, in oda-hub/dispatcher-app#263

Note that since I added this role to you, you started to recieved user approval emails. Its circumstantial, it's really Carlo who knows most people who are likely to join, and sometimes you. We should really add a new role.

It's ok, I have no problems with that.

@volodymyrss volodymyrss modified the milestones: v21.10.0002, v21.10.0003 Nov 5, 2021
@volodymyrss volodymyrss modified the milestones: v21.10.0003, v21.10.0004 Dec 1, 2021
@volodymyrss volodymyrss removed this from the v21.10.0004 milestone Jan 10, 2022
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

No branches or pull requests

2 participants