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

Relax API support (new features + bug fix) #148

Open
wants to merge 29 commits into
base: development
Choose a base branch
from

Conversation

rlaiola
Copy link
Contributor

@rlaiola rlaiola commented Jul 6, 2021

Reference issues

  1. Relax API makes use of React JS, a UI framework, to load data asynchronously on a single page application. Although this approach may work for debugging purposes, it does not favor integration with third-party applications/systems (the returned page does not include the data);
  2. In spite of the URL arguments (:source, :id, :filename and :index), on load Relax API always makes use of the first local group (local/uibk/local/0);

What does this implement/fix?

  1. Adds support to load groups synchronously. This way a third-party application will be able to process the returned page accordingly (it will include the data!);
  2. Adds logic to set the proper :source, :id, :filename and :index when loading Relax API single page application;

How to test this PR?

Test it live at https://rlaiola.github.io/relax/.

Other Comments

Feel free to suggest other approaches to address the reported issues/features, I'm not really a React JS expert. Moreover, I think that Relax API should be modified to return data only (instead of a web page). Not sure about the best way to do that, but I'm developing a workaround to address that.

@rlaiola
Copy link
Contributor Author

rlaiola commented Jul 20, 2021

In this project you can find my solution for transforming Relax Query API into an actual REST API: https://github.com/rlaiola/relax-api

github-actions bot and others added 5 commits August 20, 2022 09:29
@rlaiola rlaiola deleted the branch dbis-uibk:development October 18, 2022 16:18
@rlaiola rlaiola closed this Oct 18, 2022
@rlaiola rlaiola deleted the development branch October 18, 2022 16:18
@rlaiola rlaiola restored the development branch October 21, 2022 14:10
@rlaiola rlaiola reopened this Oct 21, 2022
@rlaiola rlaiola mentioned this pull request Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants