Skip to content

Merge Feature branches/forms into v.next #520

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

Merged
merged 61 commits into from
Jul 18, 2024
Merged

Conversation

kaushikrw
Copy link
Collaborator

@kaushikrw kaushikrw commented Jul 18, 2024

Related to issue: #

Description:

Merges the latest feature form work into v.next for the release

Summary of changes:

  • FeatureFormsApp now uses a client id that is made public for OAuth and doesn't require a user to supply it. The README is also updated to reflect this.
  • Added theming support for AttachmentsFormElement. This also includes a new test.
  • AttachmentsFormElement.attachments list is now displayed in reverse order as it sorts the attachments from newest to oldest.
  • Thumbnails for each attachment in AttachmentTile are loaded directly into memory instead of writing them to a file. This is due to a lack of a unique identifier available to create a file. More info in the issue here .
  • Empty attachments are blocked from being downloaded due to sdk limitations. This also includes blocking rename/delete.
  • Large attachments > 50 MB are blocked from being downloaded due to sdk limitations. This includes blocking a rename.

Pre-merge Checklist

Soren Roth and others added 30 commits March 18, 2024 21:36
* updated radio button tests

* added test doc link
* slight edits to FeatureForm composable function KDoc.

* Update toolkit/featureforms/src/main/java/com/arcgismaps/toolkit/featureforms/FeatureForm.kt

Co-authored-by: Gunther Heppner <[email protected]>

---------

Co-authored-by: Soren Roth <[email protected]>
Co-authored-by: Gunther Heppner <[email protected]>
* update exception name for API change

* update exception name for API change

* pull in the build of the SDK with the name change of IncorrectValueTypeException

---------

Co-authored-by: Soren Roth <[email protected]>
* use geo-compose in FeatureFormsApp.

* remove DI providing MapViewProxy.
add AndroidViewModel usage

---------

Co-authored-by: Soren Roth <[email protected]>
… range. (#376)

* do not pass a selected date into date picker if it is out of range.

* just expand the year range to incloude out of range field value's date

* add an initialError property to the DateTimePickerState, weave it through to the PickerHeader.

* use includeTime and adjust spacing.

---------

Co-authored-by: Soren Roth <[email protected]>
* wip

* basic support for attachment element state creation and management

* remove unused dependency

---------

Co-authored-by: Soren Roth <[email protected]>
@kaushikrw kaushikrw self-assigned this Jul 18, 2024
@kaushikrw kaushikrw marked this pull request as ready for review July 18, 2024 18:56
@eri9000 eri9000 self-requested a review July 18, 2024 22:48
@kaushikrw kaushikrw merged commit 780ba20 into v.next Jul 18, 2024
@kaushikrw kaushikrw deleted the feature-branches/forms branch July 18, 2024 23:10
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.

3 participants