dev - All Remotes Layout #27
Assignees
Labels
android
Related to android application.
under-development
UX
Related to user experience (i.e. graphics, animations, unresponsiveness, etc)
Comments
MattS8
added a commit
that referenced
this issue
Nov 28, 2019
#17 - Removed unused code from OLD_RemoteFragment ('determineState()', State, etc) - Creating a button no longer shows a Snackbar notification - Moved the remoteLayout's onScrollListener to MainActivity as it is now used by multiple fragments #25 - Added new Permission class. This is is used to determine what permissions the current user has for each remote. It will also allow for remote permissions management in the future. - Moved 'copyFrom' function from RemoteCreator to RemoteProfile #27 - Added xml view for an remote item - Added 'Create first remote' prompt in 'f_remote_all' - Added new 'tempRemoteName' string to TempData. This is used to store a new name for tempRemoteProfile - Developed much of 'MyRemotesFragment'. Currently shows a list of all remotes a user has access to and dynamically shows an "edit remote name" button if the user has 'READ_WRITE' permission. - Added onScrollListener that shows/hides MainViewActivity's UI views (previously unique to OLD_RemoteFragment) #28 - Added remoteListeners and remotePermissionListeners to FirestoreActions. These keep track of all the listener registrations to each remote and their accompanying permissions - Moved much of 'listenToRemote()' 's logic to two new listener classes: RemoteListener and RemotePermissionListener. Both of these handle what to do once remote data / remote permission data is received and make the original function cleaner/easier to read. - Upon receiving info for a remote, a check is now performed to see if the newly fetched data should be automatically loaded into TempRemoteProfile. This is handy for immediately loading a user's favorite remote upon first load up. - Added new 'updateRemoteName()' function that allows the user to update only the remote's name. No permission checks are done in this function! It is assume the caller has already done the necessary checks. If not, the backend may deny the change. - Rearranged the code in 'addRemote()' because I thought it made it easier to read (debatable). - Added a constant string for the Firestore "remotes" endpoint. In the future I should make all of the main endpoints utilize such a constant. - Removed old and commented out code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Issue Description:
Development of the layout or any pieces regarding showing a list of all remotes a user has access to.
The text was updated successfully, but these errors were encountered: