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.

Sign up for GitHub

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

Jump to bottom

Apis refactor #433

Merged
merged 17 commits into from
Nov 7, 2024
Merged

Apis refactor #433

merged 17 commits into from
Nov 7, 2024

Conversation

Demurest
Copy link
Contributor

@Demurest Demurest commented Nov 7, 2024

Cambios:

  • Se actualizaron los scripts de xmatch, basicInformation, classifiers y magstats, con la finalidad de utilizar las nuevas apis mediante HTMX.
  • Se añadió un plugin de HTMX que permite de visualizar los logs en la consola en un entorno distinto a producción.

Demurest and others added 17 commits August 20, 2024 14:15
@Demurest
integration magstat api htmx
e978910
@Demurest
object details api call
6d5c89d
@HectorxH
Merge branch 'develop' of github.com:alercebroker/ztf_explorer into i…
fbc17a4 
…ntegration/ApisHtmx
@HectorxH
fix: update env variables
00f3499
@HectorxH
feat: fix doubled htmx calls
3e4ccf3
@HectorxH
fix: only update htmx on loading
4e38b2a
@HectorxH
ci: use new env variable on deploy workflow
aa6a1c5
@HectorxH
Merge pull request #427 from alercebroker/integration/ApisHtmx
64aca00 
Object_details and Magstats API's
@HectorxH
ci: add old api env var
8c30692
@HectorxH
Merge pull request #428 from alercebroker/integration/ApisHtmx
f27d335 
Fix env variables
@Demurest
probability call
8bdf74e
@Demurest
call to staging api's
cc1011b
@Demurest
remove watch
3c043e6
@Demurest
Merge pull request #431 from alercebroker/feat/probability_api
41b93b2 
Probability API call Htmx
@Demurest
xmatch api call
776c369
@Demurest
remove show prop
da02a89
@Demurest
Merge pull request #432 from alercebroker/feat/xmatch
c5747fa 
xmatch api call
@Demurest Demurest self-assigned this Nov 7, 2024
HectorxH
HectorxH approved these changes Nov 7, 2024
View reviewed changes
Copy link
Contributor

@HectorxH HectorxH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Demurest Demurest merged commit 05c7eca into main Nov 7, 2024
1 check passed
@Demurest Demurest mentioned this pull request Nov 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HectorxH HectorxH HectorxH approved these changes

Assignees

@Demurest Demurest

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Demurest @HectorxH