Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

fix: upgrade nodejs and dependency versions, update README links #36

Merged
merged 1 commit into from
Mar 21, 2022
Merged

fix: upgrade nodejs and dependency versions, update README links #36

merged 1 commit into from
Mar 21, 2022

Conversation

florianm
Copy link
Contributor

  • Upgraded deps installed locally and tests are passing.
  • Upgraded Docker image built and tested locally: XLSForm exports OK.

@florianm florianm self-assigned this Mar 21, 2022
yanokwa
yanokwa approved these changes Mar 21, 2022
View reviewed changes
Copy link
Member

@yanokwa yanokwa left a comment

Choose a reason for hiding this comment

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

It'd be good to know what forms you tested. Was it a 1 question form or something with nested questions, multiple languages, etc.

@florianm
Copy link
Contributor Author

Relatively simple, four questions, 9ne language. I'll test with a more complex form today and will open a new issue if I find any bugs.

@florianm florianm merged commit f023f2b into getodk:master Mar 21, 2022
@florianm florianm deleted the dep-upgrades branch March 21, 2022 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@yanokwa yanokwa yanokwa approved these changes

@lognaturel lognaturel Awaiting requested review from lognaturel

Assignees

@florianm florianm

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@florianm @yanokwa