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

Edit user changes #247

Closed
wants to merge 3 commits into from
Closed

Edit user changes #247

wants to merge 3 commits into from

Conversation

droberts-ctrlo
Copy link
Contributor

No description provided.

droberts-ctrlo and others added 3 commits October 16, 2023 17:04
* Added development files ignore

* More info button

Added more info button, moved CSS and updated site.js to include

* Delete button

Added delete button and updated site.js

* Submit field button

Added submit field button and updated site.js

* Submit draft record button

Added submit draft record button and updated site.js

* Submit Record Button

Added submit record button and updated site.js

* Removed extra "more info" button

Removed code for extra button to avoid confusion

* Save view button

Added save view button and updated site.js

* Show blank button

Added show blank button and updated site.js

* Remove Curval Button

Added remove curval button and updated site.js

* Remove Unload Button

Added remove unload button and updated site.js

* Calculator button

Added calculator button, removed extra button component as no longer needed, and updated site.js

It is worth noting for now that in the original button component had a "hanging" function (i.e. not implemented) called `this.initCalculator()` - it is thought that this is why the calculator was inactive in v2. Discussion is required to ascertain whether implementation is required, or the button can be removed.

* Webpack Error Fix

Fixed errors stopping Webpack compilation

* Create webpack.yml

* Update webpack.yml

* Fix for minor bug in curval button and submit button

Code tested within reasonable bounds - I believe it is now ready for release

* Removed unnecessary workflow

* Fixed erroneous exclusion of package.json and yarn.lock
* Create webpack.yml

* Update webpack.yml

* Multi-field asterisk now present (I believe)

* Fixed error where required trees would not display asterisk

* Multi-field asterisk now present (I believe)

* Fixed error where required trees would not display asterisk

* Formatting fix and removed action that wasn't required

* removed webpack.yml
- Added notes field to user on edit as well as open
- Allowed for rename of notes field
- Allowed for modification of placeholder text in notes field
- Allowed for modification of placeholder text in freetext fleids
- Modified notes field to allow to be textarea
- Edited template to allow for textarea data type
@droberts-ctrlo droberts-ctrlo closed this by deleting the head repository Oct 17, 2023
@droberts-ctrlo
Copy link
Contributor Author

I messed up - I'll do a new pull!

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.

1 participant