Florida
Account created on 25 April 2007, over 18 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States swirt Florida
🇺🇸United States swirt Florida

Crediting krafferty who did a great UX rework, dramatically improving the ability to understand the settings page.

🇺🇸United States swirt Florida

I no longer think there is a reason to create archives of drafts. The flow can simply be, if the dataset state is published, it creates a published archive.

🇺🇸United States swirt Florida
🇺🇸United States swirt Florida
🇺🇸United States swirt Florida
🇺🇸United States swirt Florida
🇺🇸United States swirt Florida
🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

This is completed, I am waiting to close it until the people involved have accounts so I can credit them.

🇺🇸United States swirt Florida
🇺🇸United States swirt Florida

I attempted to make these entity references to data nodes of field_data_type keyword and theme, however this became way too complicated and no longer useful because the title of those entities are meaningless hashes and the name of the theme or keyword is buried in the json form data making it hard to get to in general and seemingly impossible to query for.

So I made the decision to keep them as plain text fields.

🇺🇸United States swirt Florida

Closing this as a duplicate of Design and add project browser icon Active

🇺🇸United States swirt Florida
🇺🇸United States swirt Florida
🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida
🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

Crediting dafeder for his help in sorting out the dependency problem on this and creating the solution in dkan.
https://github.com/GetDKAN/dkan/releases/tag/2.21.0
https://github.com/GetDKAN/dkan/issues/4545

🇺🇸United States swirt Florida

This is unblocked now that DKAN 2.21 is released.

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

Merged this.

🇺🇸United States swirt Florida

Closing this as the single directory component uswds-alert-item.component handles both the USWDS Alert and the USWDS Site Alert. This ticket is no longer needed.

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

I think that related issue is the wrong one, because it is referencing this issue.

🇺🇸United States swirt Florida

Fixing a broken link to a post that moved and was renamed.

🇺🇸United States swirt Florida

This one is ready for review.

🇺🇸United States swirt Florida

This field no longer exists. I am not certain we need this field anymore. It might be useful for connecting an archive to a dataset, but it is probably easier to use the dataset id.

Keeping this ticket open until things are more finalized.

🇺🇸United States swirt Florida

I'd like to see this take the same approach as the alt_text_validation report. There are a few benefits:

  1. The report builds on a queue so it does not tie the site up.
  2. Cron enabled by settings
  3. It will find a link wherever it exists in link fields and html text fields, not just in nodes.
  4. The report is a view that can be filtered.
  5. The report has a downloadable CSV

It is made up primarily of the following code:

  1. src/Controller/AltTextValidationAudit.php
  2. src/Service/BatchProcessor.php
  3. src/Service/Auditor.php
  4. src/Service/AuditorInterface.php
  5. src/Service/AuditStorage.php
  6. src/Plugin/QueueWorker/ImageFieldAuditor.php
  7. src/Plugin/QueueWorker/EntityImageFieldQueuer.php
  8. Views related stuff
  9. src/Drush/Commands/AltTextValidationCommands.php

It looks like a lot, but for the most part it would not need much altering to make it work.

🇺🇸United States swirt Florida

After going through this with the help of dmundra we noticed that the customizable headings for the error and warning messages are problematic for translations. That feature is of low value so it makes sense to remove that feature and replace it with properly translatable hard coded text. I create the child issue 📌 Remove custom warning and error headings Active to make it so that it does not slow down this PR.

This is looking really good. and I think will be mergeable once we get the version of the View updated in the config/install so that language appears in both the View columns and filter as well as the CSV version of the report.

🇺🇸United States swirt Florida

This has been resolved on our agency's instance of Zscaler. Apparently port 22 was on blocked on Zscaler for the destination.

Changing this to Closed (outdated).

🇺🇸United States swirt Florida

I am thinking that any of the aggregated archives need to contain and entity reference to any individual archives they contain. That way if the aggregation ever needs to be updated, there is a record of what to grab.

🇺🇸United States swirt Florida

Closing this as no longer needed. Tpp much work involved with maintaining the epic. Anything listed as Major is part of the MVP

🇺🇸United States swirt Florida

These should replicate the whitelist field on node link report settings. Comma separated list that gets alpha sorted on save.

🇺🇸United States swirt Florida

I will report back here when I get a response.

🇺🇸United States swirt Florida

Thanks @drumm I appreciate the help and will take your findings back to Zscaler and see if they can go farther.

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

Crediting those who worked on the code in the prior location.

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

Crediting those who did the work in the prior location

🇺🇸United States swirt Florida

swirt created an issue.

🇺🇸United States swirt Florida

Crediting those who did the original work.

🇺🇸United States swirt Florida

swirt created an issue.

Production build 0.71.5 2024