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.

Sign up for GitHub

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

Jump to bottom

Testing Log #1208

Open
4 of 11 tasks
GeorgeBellTMH opened this issue Jul 28, 2022 · 53 comments
Open
4 of 11 tasks

Testing Log #1208

GeorgeBellTMH opened this issue Jul 28, 2022 · 53 comments
Assignees
@laurengermain

Comments

@GeorgeBellTMH
Copy link
Contributor

GeorgeBellTMH commented Jul 28, 2022 •
edited by lucastbelem
Loading

@GeorgeBellTMH
Copy link
Contributor Author

@lucastbelem how's this going?

@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Aug 14, 2022 •
edited by lucastbelem
Loading

  • Remove groups and events
  • Add disable user
  • Remove "Show" from map toggles

@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Aug 14, 2022 •
edited by lucastbelem
Loading

  • Fixed submenu not disappearing

@lucastbelem
Copy link
Collaborator

  • Remove groups and events
  • Add disable user
  • Remove "Show" from map toggles

These are not showing in dev.

@lucastbelem
Copy link
Collaborator

  • Fixed submenu not disappearing

Also not showing.

@lucastbelem
Copy link
Collaborator

  • Replaced maps with new version of maps

I can't tell that any changes were made. When you say "new" do you mean from a number of months ago?

@lucastbelem
Copy link
Collaborator

@GeorgeBellTMH we may need to connect so I can test further...I'm not super clear on what it is I'm testing for some of these. For example, the removal of react base...how do I test for that?

@GeorgeBellTMH
Copy link
Contributor Author

I need to publish again, I tried but the up speed was so bad I stopped it. Will try again tonite.

@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Sep 13, 2022 •
edited by itbel
Loading

Published directly to both beta and dev

@lucastbelem
Copy link
Collaborator

@laurengermain

@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Sep 22, 2022 •
edited by itbel
Loading

@GeorgeBellTMH
Copy link
Contributor Author

Published to dev at 12:48am - Sept 22....please confirm testing is complete and I will send to beta...I think we want to get this out ASAP...

@laurengermain
@lucastbelem

@laurengermain
Copy link

@itbel #1258 Non partners are not appearing in the people directory and carousel so this is fixed. However, new partners are also not appearing in the directory and carousel (I tested this by adding partner permission to one of my profile's).

@laurengermain
Copy link

#1255 ready to publish.
#1259 ready to publish.
#1262 Instructors, back office staff, and admins do not see cohorts (only the learning collective section appears). Fixed for cohort users and facilitators. @itbel
#1263 ready to publish.

@GeorgeBellTMH

@itbel itbel pinned this issue Sep 23, 2022
@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Sep 27, 2022 •
edited by itbel
Loading

published on sep 27@2pm

@GeorgeBellTMH
Copy link
Contributor Author

cc: @laurengermain

@itbel
Copy link
Collaborator

itbel commented Sep 27, 2022 •
edited
Loading

published on sep 27@2pm

This can be tested by sending a direct message to another user, and ensure the user receives an email notification.
Also, ensure closed cohort email notifications are working as intended.

@itbel
Copy link
Collaborator

itbel commented Sep 29, 2022 •
edited
Loading

#1262 should be working as intended... as per @lucastbelem

@lucastbelem
Copy link
Collaborator

lucastbelem commented Sep 29, 2022 •
edited by itbel
Loading

@laurengermain as long as cohort members can see the cohort they belong to apart from the learning collective, then we're in good shape. #1262

@itbel
Copy link
Collaborator

itbel commented Sep 30, 2022 •
edited
Loading

#1258 is working as intended... Adding a user to a group does not immediately update it in dynamo. The update occurs at 9PM EST every day.

@itbel
Copy link
Collaborator

itbel commented Sep 30, 2022

You can confirm this by checking the people directory now... that user you added partner permission to should be showing up now. @laurengermain

@laurengermain
Copy link

laurengermain commented Sep 30, 2022 •
edited
Loading

#1262 ready to publish
#1264 ready to publish
#1258 ready to publish
@GeorgeBellTMH @itbel

@laurengermain
Copy link

Tested the following and all are working:
#1254
#1252
#1251
#1246

@GeorgeBellTMH
Copy link
Contributor Author

GeorgeBellTMH commented Nov 8, 2022 •
edited by lucastbelem
Loading

#1271 @laurengermain pls test.

@lucastbelem
Copy link
Collaborator

Views, headers, cards replaced with Views...check all layout and styling\n Badge switch to material from react-base\n Removal of react-base\n Version updates (fixed regression in sidebar)

@GeorgeBellTMH not sure how Lauren can test these. Can you provide instructions? Or confirm that these are in fact done?

@laurengermain
Copy link

#1271 @laurengermain pls test.

I can no longer create a new account on the sign in page: when I click, "Create an Account" the entire screens goes blank.

Screen Shot 2022-11-10 at 2 47 09 PM

@GeorgeBellTMH
Copy link
Contributor Author

@GeorgeBellTMH
Copy link
Contributor Author

Views, headers, cards replaced with Views...check all layout and styling\n Badge switch to material from react-base\n Removal of react-base\n Version updates (fixed regression in sidebar)

@GeorgeBellTMH not sure how Lauren can test these. Can you provide instructions? Or confirm that these are in fact done?

It's mainly just making sure there are no obvious formatting or style errors...

@lucastbelem
Copy link
Collaborator

@laurengermain more tests for this week.

@laurengermain
Copy link

#1277 ready to publish
#1276 ready to publish
#1273 Chrome: search by first and last name working but not by email address. Mobile version and Safari: search function appears but I can only type 1 letter into the search field.
Screen Shot 2022-11-30 at 2 19 04 PM

@lucastbelem
Copy link
Collaborator

@itbel

@lucastbelem
Copy link
Collaborator

@itbel can you pls look at #1273 again? @laurengermain identified issues.

@itbel
Copy link
Collaborator

itbel commented Dec 8, 2022

Ok

@lucastbelem
Copy link
Collaborator

lucastbelem commented Dec 8, 2022 •
edited
Loading

@laurengermain can you pls verify that separated cohorts works fully?

@itbel
Copy link
Collaborator

itbel commented Dec 8, 2022

@laurengermain can you pls verify that separated cohorts works fully?

I don't believe this was implemented...

@itbel
Copy link
Collaborator

itbel commented Dec 8, 2022

#1277 ready to publish #1276 ready to publish #1273 Chrome: search by first and last name working but not by email address. Mobile version and Safari: search function appears but I can only type 1 letter into the search field. Screen Shot 2022-11-30 at 2 19 04 PM

#1273
Search by email is working for me.
I was unable to replicate the issues you encountered using Safari and chrome mobile version. Working fine for me..

@lucastbelem
Copy link
Collaborator

lucastbelem commented Dec 8, 2022 via email

@laurengermain
Copy link

@itbel @lucastbelem I tried all of these again but no luck...
Screen Shot 2022-12-09 at 4 13 17 PM

@itbel
Copy link
Collaborator

itbel commented Dec 12, 2022

Fixed search by email in #1286 .. I'm still unable to replicate the other two issues...

@GeorgeBellTMH
Copy link
Contributor Author

#1277
#1286

@GeorgeBellTMH
Copy link
Contributor Author

published 10:30am Dec 20 to dev

@laurengermain
Copy link

#1277 ready to publish to beta
#1286 can search partners by email - ready to publish to beta
@GeorgeBellTMH

@lucastbelem
Copy link
Collaborator

Woot woot! Let's get this over to beta today @GeorgeBellTMH.
@laurengermain once George publishes, can you do a through check of all major functions? It's been months since we published so we want to make sure our publish isn't gonna break anything else before the holidays.

@GeorgeBellTMH
Copy link
Contributor Author

Published to beta.... @laurengermain

@GeorgeBellTMH
Copy link
Contributor Author

#1240
#1155

@GeorgeBellTMH
Copy link
Contributor Author

Publishing front end now

@GeorgeBellTMH
Copy link
Contributor Author

I've done a big update of amplify, updated the date/time picker controls and the dropdowns...amongst all sorts of other stuff...

@GeorgeBellTMH
Copy link
Contributor Author

I've done some updates to get building working on iOS/android...

  • Brand is only JC on mobile
  • Re-organized styles to work on mobile
  • Fixed issue with removing listeners when scaling windows (on web and mobile)
  • Fixed a bunch of warnings and other assorted issues
  • Fixed dynamic image loading issues

@GeorgeBellTMH
Copy link
Contributor Author

@itbel lets make sure we can build on iOS and Android going forward...please check before committing...

@GeorgeBellTMH
Copy link
Contributor Author

@lucastbelem it's up to you how much more we should do on this...I think it's mostly styling to get it working at this point...

@laurengermain
Copy link

laurengermain commented Jan 3, 2023 •
edited
Loading

#1240 ready to publish to beta
#1155 ready to publish to beta
@GeorgeBellTMH

@lucastbelem
Copy link
Collaborator

I've done some updates to get building working on iOS/android...

  • Brand is only JC on mobile
  • Re-organized styles to work on mobile
  • Fixed issue with removing listeners when scaling windows (on web and mobile)
  • Fixed a bunch of warnings and other assorted issues
  • Fixed dynamic image loading issues

You're talking about building it as a mobile app in the store, yeah?

@lucastbelem
Copy link
Collaborator

Issues with latest build:

  • JC logo gone from top left of nav bar
  • Blank screen appeared between permissions...I had only subscription valid and then added admin
  • Search appears to be returning results, but the dropdown with results are blank (though clickable and navigate to the correct place)
  • Directory buttons styling is wrong...text off centre (i.e. filter, sort, create)
  • Headers are gone in a particular resource

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@laurengermain laurengermain

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@itbel @GeorgeBellTMH @lucastbelem @laurengermain