Page MenuHomePhabricator

jsn.sherman
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Feb 17 2017, 7:18 PM (163 w, 8 h)
Availability
Available
LDAP User
Unknown
MediaWiki User
Jsn.sherman [ Global Accounts ]

Recent Activity

Tue, Mar 10

jsn.sherman added a comment to T247322: Platform unable to resolve database hostname as expected.

okay, hotfix made it to production branch. dropped the local changes on production host, pulled, and redeployed the stack. Production is looking happy. Staging rebase will be another day.

Tue, Mar 10, 5:17 PM · Library-Card-Platform
jsn.sherman added a comment to T247322: Platform unable to resolve database hostname as expected.

before making that change, I tried a stack redeploy, some manual network dis/reconnects, package updates, and a reboot.

Tue, Mar 10, 5:11 PM · Library-Card-Platform
jsn.sherman added a comment to T247322: Platform unable to resolve database hostname as expected.

it's not clear why the alias name spontaneously stopped resolving on both staging and production hosts.

Tue, Mar 10, 5:08 PM · Library-Card-Platform
jsn.sherman added a comment to T247322: Platform unable to resolve database hostname as expected.

hotfix posted to master. I was able to work around this issue by using the name entry used internally by the docker network manager, eg. s/db/tasks.production_db/g where production is the stack name and db is the service alias
https://github.com/WikipediaLibrary/TWLight/commit/a5c77fdbd0323114fd3c525be56e3dd7b726c6d8
I'll stash the change on prod and pull again once it makes it to the production branch. Will do the same for staging as part of the upcoming rebase.

Tue, Mar 10, 5:07 PM · Library-Card-Platform

Feb 28 2020

jsn.sherman renamed T246363: In staging all users are failing Library Bundle checks from In production all users are failing Library Bundle checks to In staging all users are failing Library Bundle checks.
Feb 28 2020, 3:54 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T246363: In staging all users are failing Library Bundle checks.

the should-be fix has been merged to staging. Should be live on the staging site upon deploy of latest staging image

Feb 28 2020, 3:54 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

merged into staging.

Feb 28 2020, 3:49 PM · Library-Card-Platform-Proxy

Feb 27 2020

jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

and now we've got Editor.ignore_wp_blocks to force Editor.wp_valid to true if blocks are the reason it's false.

Feb 27 2020, 9:16 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

Now that we've gone through all of the bundle launch blockers, I decided to see if I could do something simpler than what I had initially planned here.
This PR should now have global block checks working (they were broken previously).
The way in which the central auth extension presents its data means that we don't even have to make an extra query after all. We're just getting back a larger response from a request we're already having to make.

Feb 27 2020, 8:57 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T246363: In staging all users are failing Library Bundle checks.

40 Hours

Feb 27 2020, 5:30 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T244896: Differentiate proxy and bundle authorizations.

25 hours

Feb 27 2020, 5:30 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T244895: Drop all sessions when deploying Library Bundle.

5 hours

Feb 27 2020, 5:28 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T236287: Create a management command to email proxy/bundle users for the rollout.

5hrs

Feb 27 2020, 5:26 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

estimate 25 hours of work from here

Feb 27 2020, 5:17 PM · Library-Card-Platform-Proxy

Feb 25 2020

jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

PR in progress
https://github.com/WikipediaLibrary/TWLight/pull/413

Feb 25 2020, 8:16 PM · Library-Card-Platform-Proxy

Feb 14 2020

jsn.sherman updated the task description for T245257: Improve usability of our staging environment.
Feb 14 2020, 5:16 PM · Library-Card-Platform
jsn.sherman added a comment to T153432: Ensure that we can track how many editors are actively using authentication.

there are statistics available for download via the ezproxy reporting interface. The login details are in 1password

Feb 14 2020, 5:07 PM · Library-Card-Platform-Proxy

Feb 11 2020

jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

okay, tracked down the query for global blocks:
https://www.mediawiki.org/w/api.php?action=query&meta=globaluserinfo&guiid=<numeric_id>&format=json&guiprop=merged|unattached
It returns some really nice contextual information, such as block duration and reason. I'm going to store concatenated json of just the records that contain blocks. It's also really fast, so I think we can just let it stay synchronous after all.

Feb 11 2020, 6:41 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T242742: When a partner is switched to Library Bundle access method, set all open applications to Invalid.

ah, yeah I can see that we already have some bundle info in the about page.

Feb 11 2020, 4:46 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T242742: When a partner is switched to Library Bundle access method, set all open applications to Invalid.

The signal in this pr posts a comment to the application indicating that the partner is now part of the bundle. We discussed adding a link to the 'my collections' page which makes sense for bundle eligible editors. Should we be sending them to the terms page (or somewhere else) if they are ineligible for bundle at the time?

Feb 11 2020, 3:35 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T242742: When a partner is switched to Library Bundle access method, set all open applications to Invalid.

as a sidenote, this doesn't try to do anything clever in the case of open apps from editors who have deleted their data/accounts. Not sure if that is even a case that needs to be accounted for, but I thought I'd mention that I haven't even looked.

Feb 11 2020, 2:18 PM · Library-Card-Platform-Proxy

Feb 10 2020

jsn.sherman added a comment to T242742: When a partner is switched to Library Bundle access method, set all open applications to Invalid.

PR is up
https://github.com/WikipediaLibrary/TWLight/pull/403

Feb 10 2020, 7:49 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T236781: Track users' recent edits (and block status) for Library Bundle requirements.

note that the current code doesn't actually update block status: as we only checking it against the oauth information, which is updated at user login. Related questions in T234552 comments

Feb 10 2020, 4:25 PM · Library-Card-Platform-Proxy
jsn.sherman claimed T242742: When a partner is switched to Library Bundle access method, set all open applications to Invalid.
Feb 10 2020, 3:31 PM · Library-Card-Platform-Proxy
jsn.sherman claimed T234552: Users who are blocked on any project should require manual review for Bundle approval.

in the current code on staging, users marked as blocked from the oauth process have no way to access bundle content.

Feb 10 2020, 3:29 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T236781: Track users' recent edits (and block status) for Library Bundle requirements.

this code has been written and is live on staging. awaiting feedback.

Feb 10 2020, 3:27 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234552: Users who are blocked on any project should require manual review for Bundle approval.

As I was looking at this, I realized that we need to verify that our current check for blocks is actually a global check, not just checking for blocks on meta. Assuming we have/get a global block check in place, allowing overrides might turn out to be tricky. I'm imagining implementing this as a per-editor flag to not enforce the blocked status check for validity.

Feb 10 2020, 3:25 PM · Library-Card-Platform-Proxy

Feb 3 2020

jsn.sherman added a comment to T235262: Implement Library Bundle functionality.

Okay, I have the eligibility as described here implemented:
https://github.com/WikipediaLibrary/TWLight/pull/395
Per our off-thread conversations, we're not going to merge this to directly to master. Rather, I've force pushed this branch to staging, against which we will do dependent bundle-related PRs. Once all bundle related work is merged into staging and happy, we'll do a staging -> master PR for all bundle functionality.

Feb 3 2020, 4:34 PM · Library-Card-Platform

Jan 14 2020

jsn.sherman added a comment to T236781: Track users' recent edits (and block status) for Library Bundle requirements.

we need separate props for each requirement, account age, total edit count, not blocked and 30 day edit count

Jan 14 2020, 12:49 PM · Library-Card-Platform-Proxy

Dec 12 2019

jsn.sherman added a comment to T203163: Fix List Applications overlapping statistics for waitlisted partners with short description panels.

@HAKSOAT what version of Ubuntu are you running? Also, can you verify if your computer has a 64-bit AMD/Intel processor?

Dec 12 2019, 3:18 PM · good first task, Library-Card-Platform

Nov 20 2019

jsn.sherman added a comment to T238656: Change ToU UX and ensure users don't access proxied resources without first agreeing to the ToU .

Since the terms agreement is monolithic, we probably shouldn't allow authenticated users who haven't agreed to do anything other than agree, delete their data, or log out.

Nov 20 2019, 1:09 PM · Library-Card-Platform-Proxy
AVasanth_WMF awarded T209092: Upgrade to Python 3 a Yellow Medal token.
Nov 20 2019, 4:28 AM · Library-Card-Platform

Nov 19 2019

jsn.sherman closed T209092: Upgrade to Python 3 as Resolved.

we're now on python 3.

Nov 19 2019, 11:37 PM · Library-Card-Platform
jsn.sherman added a comment to T209092: Upgrade to Python 3.

Python 3 upgrade is merged in and wending its way through the deployment pipeline.

Nov 19 2019, 10:51 PM · Library-Card-Platform

Nov 18 2019

jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

Okay, fix deployed. I was able to access rock's backpages. If you run into the error message, try using that logout link and then try again. Let me know how it goes.

Nov 18 2019, 4:41 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

I verified that this is an EZProxy configuration issue on our end. Working on fix.

Nov 18 2019, 2:42 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

Hmm, I thought there was another error message being discussed, but I can see this is it. I don't think this is an issue with RBP, I think it's an issue with us

Nov 18 2019, 2:40 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

So, just to pitch in a little additional info:

Nov 18 2019, 2:31 PM · Library-Card-Platform-Proxy

Nov 15 2019

jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

good deal. For the record, the error definitely didn't have anything to do with your account. For whatever reason, the production system never picked up on the additional proxy configuration details that were added some weeks ago. We had do delete the docker stack and re-add it for the configuration to be available.

Nov 15 2019, 10:40 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T234612: Soft launch proxy-based access for a few partners.

@Nikkimaria This issue *should* be resolved. I verified that I was able to apply for and access MIT via proxy. Could you please verify?

Nov 15 2019, 2:59 PM · Library-Card-Platform-Proxy

Nov 12 2019

jsn.sherman claimed T237721: Optimise Metrics load time.
Nov 12 2019, 3:58 PM · Library-Card-Platform

Nov 8 2019

jsn.sherman claimed T209092: Upgrade to Python 3.
Nov 8 2019, 3:01 AM · Library-Card-Platform
jsn.sherman added a comment to T209092: Upgrade to Python 3.

Work is underway:
https://github.com/WikipediaLibrary/TWLight/pull/348

Nov 8 2019, 2:52 AM · Library-Card-Platform

Nov 7 2019

jsn.sherman closed T167817: Error during OAUTH for some users as Resolved.

I'm going to close this issue out. The underlying cause (that we can't complete the oauth process when it gets interrupted) isn't really resolvable, but we are now doing something more useful than letting an exception whiz by uncaught and sending users to a generic 500 error.

Nov 7 2019, 5:27 PM · Library-Card-Platform
jsn.sherman added a comment to T167817: Error during OAUTH for some users.

Verified that we no longer dump users out to a 500 error in this circumstance. Now we throw a 403 with a warning message:

Nov 7 2019, 5:22 PM · Library-Card-Platform
jsn.sherman added a comment to T167817: Error during OAUTH for some users.

It *looks* like this is from a user trying to complete an oauth process that got interrupted midway through. That can happen a few ways:

  • halt and continue (that is hitting 'x' during oauth, then going to the address bar and hitting enter)
  • hitting reload during the login process.
  • having a network disruption during login and then trying to continue
Nov 7 2019, 3:24 PM · Library-Card-Platform
jsn.sherman added a comment to T167817: Error during OAUTH for some users.

@AVasanth_WMF It looks like this is impacting you directly? Does this only happen intermittently? I set my client settings to be as similar to yours as possible (language set to en-GB, tried both WMF and personal handles) and was not able to reproduce. Clearly a problem is occurring as evidenced by your reports and the error logs. If the issue is intermittent, I suspect that it's due to meta not completing the oauth process occasionally, or perhaps a network interruption. Your browser is what makes the requests between the Oauth IdP on meta and the Oauth SP on twl (eg. there is no backend server <-> server communication for this part of the process). Do you have any interesting browser configuration that you could turn off for a while? or have you been experiencing intermittent network problems?

Nov 7 2019, 2:43 PM · Library-Card-Platform

Nov 4 2019

jsn.sherman added a comment to T167817: Error during OAUTH for some users.

hopeful fix deployed. Let's see if this goes away.

Nov 4 2019, 2:58 PM · Library-Card-Platform
jsn.sherman added a comment to T167817: Error during OAUTH for some users.

It looks like we're not getting a request_token from the oauth initialization. Upon researching this further, it looks like there are several ways in which the mediawiki oauth implementation deviates from standards, and it also appears to have some limitations in dealing with unicode data from service providers. It looks like there are situations in which initialization will silently fail for non-english users due to one of these unicode issues. We use the mwoauth package as a shim that works around many of these issues, and it looks like they released an update 5 days ago that might fix this particular issue. The update is currently wending its way through the deployment pipeline

Nov 4 2019, 2:26 PM · Library-Card-Platform

Oct 22 2019

jsn.sherman closed T236085: Fix authorizations with no authorizer as Resolved.

This is complete. The total number of applications without a populated sent_by was 5389.

Oct 22 2019, 5:28 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T236085: Fix authorizations with no authorizer.

so, we decided on slack to add a system user.
The steps to resolve this will look like:

Oct 22 2019, 1:55 PM · Library-Card-Platform-Proxy
jsn.sherman updated subscribers of T236085: Fix authorizations with no authorizer.

There are about 2500 imported sent apps for which the partner has no coordinator. Should I just set sent_by to @Nikkimaria in that case? Or I could create a system/statistical user for these historical cases where we can't know

Oct 22 2019, 11:21 AM · Library-Card-Platform-Proxy

Oct 21 2019

jsn.sherman added a comment to T233508: Link and Proxy authorization method partners aren't recording authorizer when authorizations are created..

hotfix deployed, keeping open until we get some verification that it's definitely fixed all known cases.

Oct 21 2019, 8:04 PM · Library-Card-Platform
jsn.sherman added a comment to T236085: Fix authorizations with no authorizer.

based on the work I did on T233508 , I think this issue will best be addressed by looping through sent applications and setting sent_by to the latest reviewer (the step that was missing on affected applications), then I can loop through applications with no authorizer and fill it in with the sent_by data as appropriate. That should catch many affected applications and authorizations.

Oct 21 2019, 8:02 PM · Library-Card-Platform-Proxy
jsn.sherman added a comment to T233508: Link and Proxy authorization method partners aren't recording authorizer when authorizations are created..

deleted the backfilled auths with this issue

Oct 21 2019, 7:45 PM · Library-Card-Platform
jsn.sherman added a comment to T233508: Link and Proxy authorization method partners aren't recording authorizer when authorizations are created..

hotfix in the pipeline

Oct 21 2019, 7:23 PM · Library-Card-Platform
jsn.sherman claimed T233508: Link and Proxy authorization method partners aren't recording authorizer when authorizations are created..
Oct 21 2019, 4:40 PM · Library-Card-Platform
jsn.sherman claimed T236085: Fix authorizations with no authorizer.
Oct 21 2019, 4:40 PM · Library-Card-Platform-Proxy

Oct 17 2019

jsn.sherman added a comment to T175573: List Applications button should show whitelist for authentication-based partners.

So we talked about this in the weekly meeting, and the only need we can think of for having this is that this is a place where coordinators can see applicant email addresses. We talked about adding a show/hide email address toggle in the review and application views to serve that need.

Oct 17 2019, 4:19 PM · Library-Card-Platform-Proxy

Oct 10 2019

jsn.sherman added a comment to T175573: List Applications button should show whitelist for authentication-based partners.

My thought is that we should just show authorizations and not applications at all. This would be implemented after we backfill missing authorizations.

Oct 10 2019, 4:55 PM · Library-Card-Platform-Proxy

Sep 5 2019

jsn.sherman claimed T225915: Fix incorrect string value error for iPhone user agent.
Sep 5 2019, 3:49 PM · Library-Card-Platform

Sep 3 2019

jsn.sherman closed T226793: UnicodeDecodeError when applying for resources as Resolved.

This was a hangover from an accidental encoding regression that occurred when we migrated to docker. It's been resolved with a combination of config, code, and ops.

Sep 3 2019, 5:45 PM · Library-Card-Platform

Aug 27 2019

jsn.sherman added a comment to T226617: UnicodeEncodeError on OAuth callback.

deployed. Let's wait till users confirm things are working to close this out

Aug 27 2019, 1:19 PM · Library-Card-Platform

Aug 22 2019

jsn.sherman added a comment to T226617: UnicodeEncodeError on OAuth callback.

Okay, I created a PR with code that seems to resolve the auth limbo issue, but it wouldn't be bad for @Samwalton9 to take a peek at it since it touches auth.

Aug 22 2019, 9:24 PM · Library-Card-Platform
jsn.sherman added a comment to T226617: UnicodeEncodeError on OAuth callback.

I've got the underlying issue resolved.
*However*
The user accounts that encountered this are in a half-created state that in practice will keep the user in authentication purgatory until we make the login process a little more robust or delete the affected accounts.
I'm going to try the coding option first, as we have some other non-operational accounts that were created for statistical purposes, and I don't want to have to try to identify just the right set of strange looking accounts to delete.

Aug 22 2019, 8:03 PM · Library-Card-Platform
jsn.sherman claimed T226617: UnicodeEncodeError on OAuth callback.
Aug 22 2019, 7:17 PM · Library-Card-Platform

Jun 13 2019

jsn.sherman added a comment to T203685: Ensure that logs are saved even when the environment is recreated.

If we don't need to keep them around for a super long time, then we could just use an external event/log analytics service like https://timber.io/ or https://sentry.io/

Jun 13 2019, 3:18 PM · Library-Card-Platform
jsn.sherman added a comment to T203685: Ensure that logs are saved even when the environment is recreated.

still an issue post-dockerization

Jun 13 2019, 3:16 PM · Library-Card-Platform
jsn.sherman added a comment to T200520: Log access to personal data.

This is still an outstanding task post-dockerization

Jun 13 2019, 2:37 PM · Library-Card-Platform
jsn.sherman closed T209838: Add a new script or flag for reloading gunicorn or running collectstatic when required as Resolved.

This was closed when we moved to docker.

Jun 13 2019, 2:36 PM · Library-Card-Platform

Jan 10 2019

jsn.sherman added a comment to T201159: Add a Contact Us page.

+1

Jan 10 2019, 6:51 PM · Library-Card-Platform
jsn.sherman added a comment to T201159: Add a Contact Us page.

I haven't paid to much attention to this until looking at @AVasanth_WMF 's excellent mockup this morning.
From a spaminess perspective, I'd recommend that we not have a public/anonymous form that sends email.

Jan 10 2019, 3:38 PM · Library-Card-Platform

Jan 8 2019

jsn.sherman closed T213007: Library Card platform isn't sending any emails - all 'Draft' as Resolved.

This has been resolved. It turns out that our underlying email infrastructure changed, but we weren't aware of it. All withheld emails should now be delivered.

Jan 8 2019, 4:50 PM · Library-Card-Platform

Jan 3 2019

jsn.sherman closed T209836: Improve user check behaviour in virtualenv_activate as Resolved.

fix merged into master

Jan 3 2019, 7:36 PM · good first task, Library-Card-Platform
jsn.sherman added a comment to T209836: Improve user check behaviour in virtualenv_activate.

So, while fixing this problem in a branch, I encountered/reproduced the originally reported issue. The scripts that directly use a virtualenv do in fact have the user check thanks to the virualenv_activate script. Some of the wrapper scripts that compose those into useful workflows don't because the test for the "right thing" needs to be a little more complicated.

Jan 3 2019, 5:25 PM · good first task, Library-Card-Platform
jsn.sherman added a comment to T209836: Improve user check behaviour in virtualenv_activate.

This is a super simple fix. I'll get it pushed right in.

Jan 3 2019, 3:45 PM · good first task, Library-Card-Platform

Dec 4 2018

jsn.sherman closed T211136: Wikipedia Library Server Error as Resolved.

hotfix in place. verified that things are happy.

Dec 4 2018, 7:44 PM · Library-Card-Platform
jsn.sherman claimed T211136: Wikipedia Library Server Error.

pushed hotfix in git. Will get it to live site asap

Dec 4 2018, 7:34 PM · Library-Card-Platform

Oct 31 2018

jsn.sherman added a comment to T208413: Check whether twl project requires NFS or not.

Yep, we definitely need nfs, that's where we keep a month's worth of nightly backups, which we've occasionally used to roll back a bad data migration or just as a place to keep our state when we throw away a vm and provision a new one.

Oct 31 2018, 7:52 PM · The-Wikipedia-Library, Cloud-VPS

Oct 19 2018

jsn.sherman added a comment to T207455: Request creation of library-card-test VPS project.

I'm planning on separating the different bits of our platform into containers running on a small kubernetes cluster, and running the database as a vm outside the cluster. We haven't determined the best way to configure persistent storage (e.g. user uploaded attachments). If we can use a project-wide nfs export (which we only use for backups in twl) that's great. If that's a performance/workload nonstarter, then I'm open to guidance, including just running a container that provides storage, which backs itself up to a project nfs export.

Oct 19 2018, 4:00 PM · cloud-services-team (Kanban), Cloud-VPS (Project-requests)

Aug 1 2018

jsn.sherman added a comment to T196573: Create production git branch and give Travis ability to push.

This is done

Aug 1 2018, 5:15 PM · Library-Card-Platform

Jul 19 2018

jsn.sherman added a comment to T196888: Comment sections are blocked by some extensions (e.g. 1Blocker).

A few thoughts:
I'm not even sure if we should fix this, since blocking comments is one of the design goals of this addon. @Xover might be interested to know that the makers of 1blocker have also released 1blocker x, which appears to be designed to address this concern for their users.

Jul 19 2018, 1:58 PM · Library-Card-Platform

Jul 17 2018

jsn.sherman added a comment to T147465: RTL-compatible styling for the Library Card tool.

Yeah, it's because the username is part of a larger block of text that's marked for translation.
https://github.com/WikipediaLibrary/TWLight/blob/b249eb48b0af54f8c0e53b3500495c452216aea5/TWLight/templates/base.html#L76

Jul 17 2018, 7:43 PM · Library-Card-Platform, RTL, I18n

Jul 13 2018

jsn.sherman added a comment to T147465: RTL-compatible styling for the Library Card tool.

@Samwalton9 I've got the (very) partial translation with the current RTL implementation up at
https://twlight-staging.wmflabs.org/

Jul 13 2018, 8:15 PM · Library-Card-Platform, RTL, I18n
jsn.sherman added a comment to T147465: RTL-compatible styling for the Library Card tool.

The tags were running over the bullets because the bullets weren't in the right place. The background-position css attribute must be explicitly set for CSS janus to modify the position of those images. Default is top left. That issue is now fixed in the dev branch.

Jul 13 2018, 7:51 PM · Library-Card-Platform, RTL, I18n
jsn.sherman added a comment to T147465: RTL-compatible styling for the Library Card tool.

Okay, I've got the issue with the empty values resolved so that we're defaulting to English when there's no suitable translation. The biggest issue I now see is that our pipe delimited tag list is running into the blue tag bullet when there are a lot of tags. Working on it.

Jul 13 2018, 7:32 PM · Library-Card-Platform, RTL, I18n
jsn.sherman added a comment to T147465: RTL-compatible styling for the Library Card tool.

@Samwalton9 In a dev branch, I've added the partial Arabic translation and fixed up numerous issues that were apparent to me both on the RTL rendering side and the translatewiki "add languages from the translation files" side. I can see that we have the additional issue if we're missing a translation for tags or descriptions, we're not displaying anything, rather than defaulting to the English version.

Jul 13 2018, 6:46 PM · Library-Card-Platform, RTL, I18n

Jul 11 2018

jsn.sherman added a comment to T198414: Scheduled coordinator reminder emails aren't being sent.

The fix for number 1 has been pushed to master, along with a fix related to number 2. In this case, we really needed to be testing the management command that creates the reminder email signal, as that's where the problem was. Let's keep this open until we see reminder emails coming down the wire.

Jul 11 2018, 8:56 PM · Library-Card-Platform

Jul 6 2018

jsn.sherman added a comment to T198414: Scheduled coordinator reminder emails aren't being sent.

I've deployed the fix for item #3.

Jul 6 2018, 8:27 PM · Library-Card-Platform

Jun 29 2018

jsn.sherman added a comment to T198414: Scheduled coordinator reminder emails aren't being sent.

@Samwalton9 These emails are still scheduled. There are 3 issues that together caused them to stop working without our knowledge.

  1. I've modified the ListApplicationsView in such a way that the reminder email template no longer works.
  2. As we've previously identified, we have inadequate test coverage for emails, so the new builds didn't start failing after the change.
  3. We're not alerting on failed cron tasks, so the system didn't tell us that it wasn't successfully completing its housekeeping as designed.
Jun 29 2018, 6:26 PM · Library-Card-Platform

May 2 2018

jsn.sherman added a comment to T193507: Re-add renewals field data.

looks like I could be doing this more efficiently:
https://django-import-export.readthedocs.io/en/latest/index.html

May 2 2018, 4:55 PM · Library-Card-Platform
jsn.sherman added a comment to T193507: Re-add renewals field data.

just keep me posted, as it's quite time consuming to reformulate the inserts from a dump into non-destructive updates. Let's try to get any other missing data identified so that we can catch it in one pass.

May 2 2018, 4:45 PM · Library-Card-Platform

Apr 30 2018

jsn.sherman closed T193333: Error on editing users as Resolved.

Issue resolved. there was a new CSRF setting that didn't come into play in local dev and travis because of the lack of SSL there.

Apr 30 2018, 3:01 PM · Library-Card-Platform
jsn.sherman added a comment to T193333: Error on editing users.

It's looking like it related to the fact that we're behind a proxy for the labs infrastructure. The issue doesn't happen in local dev, but it's there for prod and staging. I've verified that csrf is working correctly for non-admin forms.

Apr 30 2018, 1:51 PM · Library-Card-Platform

Apr 16 2018

jsn.sherman added a comment to T185244: Scope coordinator permission to be per-partner.

This change has been pushed to master, and should get picked up at tomorrow's scheduled update.

Apr 16 2018, 7:35 PM · Library-Card-Platform
jsn.sherman added a comment to T190431: Application comments not displaying on iOS.

I suppose we should mark this as closed since I have resolved all related display issues I could produce on iOS. I haven't heard back from the reporting editor, though.

Apr 16 2018, 7:34 PM · Library-Card-Platform

Apr 5 2018

jsn.sherman added a comment to T185244: Scope coordinator permission to be per-partner.

1st cut of the app code is complete. I'm in the process of extending the test suite to accommodate the way the code works now. Right now the test suite creates partners and coordinators in unrelated way. I'll need to designate coordinator/partner relationships in the test suite so that I can separately test the cases of display for a coordinator that should or shouldn't have access to a given bit of content.

Apr 5 2018, 5:30 PM · Library-Card-Platform
jsn.sherman added a comment to T190431: Application comments not displaying on iOS.

In addition to our recent css updates, I added a 400 error template, as that was the most interesting situation I was able to create while trying to break discussions. I tested on a recently acquired iphone and application discussions seem to work as expected. Awaiting feedback from the user that reported the issue.

Apr 5 2018, 3:19 PM · Library-Card-Platform

Apr 3 2018

jsn.sherman added a comment to T187486: Pageviews download as csv button doesn't work, maybe others.

Yeah, we could probably be using requests a little smarter. It returns the status code of the logged http object...which means that you get a 404 when clicking on a 404 request object. We've got some context on that buried in closed phab task somewhere.

Apr 3 2018, 4:32 PM · Library-Card-Platform

Mar 6 2018

jsn.sherman added a comment to T170795: Allow coordinators to download a file containing username, email, and application/approval date for their partner.

I'm currently evaluating using a report builder of some kind to meet this and similar needs.

Mar 6 2018, 6:46 PM · Library-Card-Platform