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

FORMS-984: Updated os2forms_rest_api exposing new list submissions en… #233

Merged
merged 2 commits into from
Oct 25, 2023

Conversation

jekuaitk
Copy link
Contributor

@jekuaitk jekuaitk commented Oct 24, 2023

…dpoint

https://jira.itkdev.dk/browse/FORMS-984

  • Updates to os2forms/os2forms_rest_api 2.0.1 exposing new list submission endpoint

@jekuaitk jekuaitk force-pushed the feature/FORMS-984-rest-api-expansion branch from 8636035 to 391d0a8 Compare October 24, 2023 13:19
@@ -10,7 +10,7 @@ dependencies:
module:
- os2forms_rest_api
_core:
default_config_hash: mjZJshmbsw6KcJln7jSGEcD087b6I0GE3wVqzEQ2MI8
default_config_hash: i6sBhZz5RGoGa-jEJNfhoVlPndf0NZQFF8L2qqg2b-Q
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure what this hash actually is, but I would be careful with changing it in existing config. Is the change a result of only drush config:export or a result of running the new update hook in the REST API module?

Copy link
Contributor Author

@jekuaitk jekuaitk Oct 24, 2023

Choose a reason for hiding this comment

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

It was a result of drush config:export after running the new update hook.

Removed config files where the only change was default_config_hash.

@jekuaitk jekuaitk force-pushed the feature/FORMS-984-rest-api-expansion branch from 391d0a8 to ac6cd43 Compare October 24, 2023 14:01
@jekuaitk jekuaitk merged commit f56683d into develop Oct 25, 2023
8 checks passed
@jekuaitk jekuaitk deleted the feature/FORMS-984-rest-api-expansion branch October 25, 2023 07:04
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.

2 participants